-
Notifications
You must be signed in to change notification settings - Fork 295
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
v1.18: blockstore: relax parent slot meta check for clear_unconfirmed_slot #68
Conversation
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.
I'm good with this minimized BP for v1.18. The previous code could panic from this previous expect:
.expect("Unconfirmed slot should have had parent slot set");
which is now an error message. While we shouldn't hit the panic in regular operation, it is technically recoverable since ReplayStage
(the caller of this function) is going to repair the slot it just purged, which will in turn populate the parent slot meta that was unable to be found.
I'm going to add @willhickey as a reviewer; let's confirm that Will is onboard with the BP as well before pushing
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## v1.18 #68 +/- ##
=========================================
- Coverage 81.6% 81.6% -0.1%
=========================================
Files 827 827
Lines 224306 224311 +5
=========================================
- Hits 183128 183117 -11
- Misses 41178 41194 +16 |
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.
Approved for beta
in the future, try to make changes with potential backportability in mind. it is much safer to take backports that are unmodified when possible |
…_slot (anza-xyz#68) blockstore: relax parent slot meta check for clear_unconfirmed_slot
backport of the relevant parts in solana-labs#35124