-
Notifications
You must be signed in to change notification settings - Fork 36.2k
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
Some database/-related recovery improvements #2558
Conversation
sipa
commented
Apr 23, 2013
- After clean shutdown, delete the database/ subdir. This means upgrading/changing the BDB dir won't kill you anymore because of incompatible logs.
- In case opening the database environment fails, move database/ to database.$TIMESTAMP.bak, and try again.
- Do not advise the user to nuke his datadir anymore - there's no point in doing so (closes BDB failures still suggest user to delete everything #2488).
- Drop double opening of the environment (includes remove duplicate bitdb.Open() code from init #2333).
- remove code from step 7, which we already have in step 5 of init
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/ccda03b57022369c4352d0f5a816cff9ace833e6 for binaries and test log. |
ACK |
Looks good and should make it for 0.8.2 IMHO. |
ACK code I'd say it needs testing on Windows, to make sure the looks-right-on-Unix code works there. Or maybe I'm just being paranoid. |
ACK. Compiles and works as advertised on OSX. |
Some database/-related recovery improvements
Some database/-related recovery improvements