Update docs/how-to/backup-your-databases.md #806

Closed
axel wants to merge 1 commits from (deleted):docs-databases-sqlite-info into main
Contributor

Make it clearer that sqlite databases are dumped and therefore require as much extra disk space available than the target database takes. Not that this bit me or anything with a silly large 43 Gb sqlite database that shouldn't even exist in the first place.

Make it clearer that sqlite databases are dumped and therefore require as much extra disk space available than the target database takes. Not that this bit me or anything with a silly large 43 Gb sqlite database that shouldn't even exist in the first place.
axel added 1 commit 2023-12-30 23:48:13 +00:00
Make it clearer that sqlite databases are dumped and therefore require as
much extra disk space available than the target database takes.
witten requested changes 2023-12-30 23:59:48 +00:00
witten left a comment
Owner

Thanks for doing this one as well. I actually don't think there's any good reason for the borgmatic SQLite hook to dump to an intermediate file. I'd be happy to merge this for now (after the requested capitalization changes!), but I've filed #807 to encompass the work of making SQLite dumps stream directly to Borg.

Thanks for doing this one as well. I actually don't think there's any good reason for the borgmatic SQLite hook to dump to an intermediate file. I'd be happy to merge this for now (after the requested capitalization changes!), but I've filed #807 to encompass the work of making SQLite dumps stream directly to Borg.
@@ -65,0 +62,4 @@
⚠️ The exceptions are the PostgreSQL/MongoDB "directory" dump formats, which
can't stream, as well as sqlite databases which are dumped. These therefore do
consume temporary disk space. (As much as the target database, for sqlite).
Owner

In this line and the previous: "sqlite" -> "SQLite".

Edit: Also ")." -> ".)"

In this line and the previous: "sqlite" -> "SQLite". Edit: Also ")." -> ".)"
Owner

Some good news and some bad news. The bad news is I implemented #807, so this PR is now out of date. The good news is I implemented #807, so SQLite dumps will no longer use an intermediate file and fill up your disk!

Some good news and some bad news. The bad news is I implemented #807, so this PR is now out of date. The good news is I implemented #807, so SQLite dumps will no longer use an intermediate file and fill up your disk!
Author
Contributor

Sorry, slow reply.
Ah ha, excellent. Well that just good news for everyone really (and the other piece of good news for me is that i no longer need to backup a 43 GB sqlite database, which is really a lot nicer).
I'll try to update the PR.

Sorry, slow reply. Ah ha, excellent. Well that just good news for everyone really (and the other piece of good news for me is that i no longer need to backup a 43 GB sqlite database, which is really a lot nicer). I'll try to update the PR.
Owner

Sounds good!

Sounds good!
Owner

Do you still plan to work on this? If not, just let me know. Thanks!

Do you still plan to work on this? If not, just let me know. Thanks!
Owner

Closing this for now due to inactivity, but I'd be happy to revisit if you'd like to reopen this.

Closing this for now due to inactivity, but I'd be happy to revisit if you'd like to reopen this.
witten closed this pull request 2024-06-24 17:49:56 +00:00

Pull request closed

Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#806