-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add consistency check at node bootup #62
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
Conversation
0a157d1 to
0dfca67
Compare
0dfca67 to
fde8739
Compare
| self.tx_ref().get::<JournalHashes>(ru_height).map_err(Into::into) | ||
| } | ||
|
|
||
| #[track_caller] |
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.
TIL about the track_caller attribute 👍
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.
yeah, it's nice. helps with diagnosing panics
dylanlott
left a comment
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.
Database and static file consistency logic looks good to me. Approving to move this along to testing, since the to do's are reproduced from reth codebase and should be ignored.
Co-authored-by: dylan <dylan@init4.technology>

Adds consistency checks and static-file healing at node bootup. This should prevent and fix storage consistency errors