-
Notifications
You must be signed in to change notification settings - Fork 458
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reduce TxPool allocations #6970
Conversation
src/Nethermind/Nethermind.TxPool/Collections/TxDistinctSortedPool.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good, but I want @marcindsobczak to review this as well.
How much allocation does it save? All the marked ones?
src/Nethermind/Nethermind.TxPool/Collections/TxDistinctSortedPool.cs
Outdated
Show resolved
Hide resolved
src/Nethermind/Nethermind.TxPool/Collections/TxDistinctSortedPool.cs
Outdated
Show resolved
Hide resolved
src/Nethermind/Nethermind.TxPool/Collections/TxDistinctSortedPool.cs
Outdated
Show resolved
Hide resolved
Yep |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approve from me, but please don't merge until @marcindsobczak reviews!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Took me a while to understand what is going on here, but actually it is just delegating low level update logic to be triggered from TxPool
instead of TxDistinctSortedPool
. I don't see any inconsistencies
Changes
Removes the following allocations
Types of changes
What types of changes does your code introduce?
Testing
Requires testing