-
Notifications
You must be signed in to change notification settings - Fork 117
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
change(state): Support in-place disk format upgrades for major database version bumps #8748
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.
This is looking great, thank you!
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.
There is a small fix required for Windows.
I was confused by some of the code but I think I got it. I suggested some comments which should hopefully help whoever reads the code in the future, feel free to tweak them.
Also I noticed the upgrade itself is not here, was it planned to be done in another PR?
Co-authored-by: Conrado Gouvea <conrado@zfnd.org>
I originally had similar comments in the code, but I deleted them because I thought the |
Yes, this PR only moves the old db to a new path. Zebra then uses the db and rewrites the The new |
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.
Ah, got it, thanks. LGTM!
Motivation
Close #8745.
Solution
Tests
PR Author's Checklist
PR Reviewer's Checklist