Skip to content
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

feat: Online backup #1265

Merged
merged 12 commits into from
Mar 15, 2023
Merged

feat: Online backup #1265

merged 12 commits into from
Mar 15, 2023

Conversation

LexLuthr
Copy link
Collaborator

@LexLuthr LexLuthr commented Mar 7, 2023

Fixes #1235

  • Add a simple test or figure out another method to test (would like to avoid itest)

@LexLuthr LexLuthr marked this pull request as ready for review March 9, 2023 12:26
node/impl/backupmgr/backup.go Outdated Show resolved Hide resolved
node/impl/backupmgr/backup.go Outdated Show resolved Hide resolved
node/impl/backupmgr/backup_test.go Outdated Show resolved Hide resolved
node/repo/boostrepo.go Show resolved Hide resolved
cmd/boostd/backup.go Show resolved Hide resolved
db/db.go Outdated Show resolved Hide resolved
@LexLuthr
Copy link
Collaborator Author

@dirkmc Can you review again after the new commits? I will be merging after this review.
@TippyFlitsUK Tested the new code and everything is working as expected.

@LexLuthr LexLuthr merged commit 8f84297 into main Mar 15, 2023
jacobheun pushed a commit that referenced this pull request Mar 17, 2023
* first version: automated online backup

* manual online backup

* register different drivers

* fix failing itests

* fix nil pntr for backup API

* fix gen

* implement suggestions

* redesign sql backup

* fix bug, restructure funcs, remove unused

* workaround failed move
@LexLuthr LexLuthr deleted the online-backup branch July 20, 2023 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Online backup for Boost
2 participants