-
Notifications
You must be signed in to change notification settings - Fork 464
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
Upgrade to RocksDB v9.4.0 #7300
Conversation
@kamilchodola Could you please compare this to the current one? |
For future there is nice action which can compare it quite easily (create two equals envs for two nodes and spawn some rpc load tests to see any difference and then can be verified on Grafana) HOPE IT STILL WORKS :) |
@kamilchodola can we merge it? |
I did not had any objections merging it last time, everything looked like working smoothly. We could merge it and keep an eye on that while testing other things. |
Memory used was reduced, no significant improvement on block processing. |
Changes
Upgraded RocksDB to v9.4.0.
Types of changes
What types of changes does your code introduce?
Testing
Requires testing
If yes, did you write tests?
Notes on testing
Needs manual testing