-
-
Notifications
You must be signed in to change notification settings - Fork 289
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: improve regen state #7033
fix: improve regen state #7033
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## unstable #7033 +/- ##
============================================
- Coverage 49.25% 49.17% -0.08%
============================================
Files 578 578
Lines 37443 37502 +59
Branches 2172 2171 -1
============================================
Hits 18441 18441
- Misses 18962 19021 +59
Partials 40 40 |
Performance Report✔️ no performance regression detected Full benchmark results
|
latest update is to load block reprocess time is around 120ms which is similar to processing a regular block. This is on feat4 mainnet node (with minimal nHistorical state config so that it has to reload state + reprocess blocks frequently) originally it is ~375ms as in #7002 |
* fix: improve regen state * fix: check for null block returned from db * feat: track state.hashTreeRoot() in regen.getState() * fix: transfer cache when regen state * fix: add caller as label to regenGetState metrics
🎉 This PR is included in v1.22.0 🎉 |
Motivation
Description
Promise.all()
, see Performance issue regen state #7002regen.getState()
:will have follow-up PR for Grafana dashboard once this is merged
closes #7002