persistent-sqlite: support for database backup #172

Closed
mikesteele81 opened this Issue Oct 11, 2013 · 5 comments

Comments

Projects
None yet
3 participants
@mikesteele81

Would you accept a pull request to add support for Sqlite's backup api to persistent-sqlite?

I'd like to provide an online save/restore feature to a Yesod application. It seems this might be possible if we had Database.Sqlite.load and Database.Sqlite.save functions.

@gregwebs

This comment has been minimized.

Show comment
Hide comment
@gregwebs

gregwebs Oct 11, 2013

Member

It probably makes sense to see if you can start this as a separate module, but we aren't necessarily opposed to including database specific functionality in the database modules.

Member

gregwebs commented Oct 11, 2013

It probably makes sense to see if you can start this as a separate module, but we aren't necessarily opposed to including database specific functionality in the database modules.

@snoyberg

This comment has been minimized.

Show comment
Hide comment
@snoyberg

snoyberg Oct 13, 2013

Member

I'm not even aware of the backup API, would you mind sharing a link to it? And I have no objection to such a pull request.

Member

snoyberg commented Oct 13, 2013

I'm not even aware of the backup API, would you mind sharing a link to it? And I have no objection to such a pull request.

@mikesteele81

This comment has been minimized.

Show comment
Hide comment
@mikesteele81

mikesteele81 Oct 14, 2013

http://www.sqlite.org/backup.html

It's a small handful of functions that lets us copy live data without locking the file.

http://www.sqlite.org/backup.html

It's a small handful of functions that lets us copy live data without locking the file.

@gregwebs gregwebs added the Sqlite label Aug 4, 2014

@gregwebs

This comment has been minimized.

Show comment
Hide comment
@gregwebs

gregwebs Aug 29, 2014

Member

closing for now, still looking forward to a pull request.

Member

gregwebs commented Aug 29, 2014

closing for now, still looking forward to a pull request.

@gregwebs gregwebs closed this Aug 29, 2014

@mikesteele81

This comment has been minimized.

Show comment
Hide comment
@mikesteele81

mikesteele81 Aug 30, 2014

I ended up not needing this feature at the time. Maybe someday I'll come back to it. Congratulations and thanks for 2.0 release, by the way.

I ended up not needing this feature at the time. Maybe someday I'll come back to it. Congratulations and thanks for 2.0 release, by the way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment