-
-
Notifications
You must be signed in to change notification settings - Fork 343
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
FIx archive finalized states db entry key typo #4508
Conversation
Performance Report✔️ no performance regression detected Full benchmark results
|
Deployed to feat2 and checked lg1k logs, which confirmed that states are pruned on start
However, metrics do not confirm that disk space is reduced, not sure why |
Due to level-db internals, after a significant amount of data put or del it's necessary to compact that range to realize the size reduction. After an offline discussion with other teams:
|
73f26a4
to
4c71288
Compare
4c71288
to
c48e83c
Compare
|
Deployed this branch to all feat2, in
After some of those the node suddenly crashes with no apparent reason |
Can we split this into two PRs:
|
Yes makes sense 👍 |
Motivation
Archive finalized states logic has a typo where it indexes states based on epoch instead of slot. That causes the archiver to not prune states as expected thus Lodestar DB grows faster than it should.
Description
Opening as draft to deploy and test before merging
Closes #4378
Closes #4417