mirror of
https://github.com/nspcc-dev/neofs-node.git
synced 2026-03-01 04:29:10 +00:00
Aggregate multiple settlement transfers into single transaction #342
Labels
No labels
I1
I2
I3
I4
S0
S1
S2
S3
S4
U0
U1
U2
U3
U4
blocked
bug
config
dependencies
discussion
documentation
enhancement
enhancement
epic
feature
go
good first issue
help wanted
neofs-adm
neofs-cli
neofs-cli
neofs-cli
neofs-ir
neofs-lens
neofs-storage
neofs-storage
performance
question
security
task
test
windows
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
nspcc-dev/neofs-node#342
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @alexvanin on GitHub (Oct 8, 2021).
During the settlement, alphabet nodes generate many transfer invocations of balance contract. Instead of sending transaction per transfer, Alphabet node can try to group multiple transfers into single transaction.
@fyrchik commented on GitHub (Jan 11, 2022):
This requires some refactoring, as we have multiple clients emdedded in each other.
One way to approach this is to have dummy client which accumulates multiple invoked scripts in a single one and then calculates GAS cost and sign/send tx in another method.
I think this is better to be done after cleaning up code for notary-disabled environment.
@roman-khimov commented on GitHub (Dec 25, 2025):
Can be easily aggregated by owner, btw. balance contract can get a list of relevant containers and loop through them, so it's like
settleOwnerPaymentinstead ofsettleContainerPayment.@roman-khimov commented on GitHub (Dec 25, 2025):
Although owner can have quite a number of containers in which case we could have a problem with system fee.