-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
perf(storage, mdbx): set rp augment limit #5614
Conversation
b7a4c47
to
a3eaa03
Compare
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.
omfg
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.
hehe
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.
send it
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.
1 loc change, 25 comments, 1 month of effort, gg
this does mean the db will grow faster on heavy freelist-related scenarios?
Yes, but this effect should be small, because only large insertions will lead to new allocations. |
Resolves #5228
Explanation
reth/crates/storage/db/src/implementation/mdbx/mod.rs
Lines 102 to 127 in a3eaa03
Results
Before 12:10 – default
rp augment limit
value (which is around137 million
)After 12:25 – custom
rp augment limit
value set to256 * 1024
Note: disregard the drop of the
Freelist
chart, it's due to node having a slight downtime during the restart and many blocks being inserted at once.