Should shell `.dump` use a transaction?

tursodatabase / turso-cli

Command line interface to Turso.

https://turso.tech

MIT License

224 stars 37 forks source link

Should shell `.dump` use a transaction? #456

Open CodingDoug opened 1 year ago

CodingDoug commented 1 year ago

sqlite3 .dump wraps the entire SQL output (after PRAGMA foreign_keys=OFF;) in BEGIN TRANSACTION; and COMMIT;. Turso's shell does not. Should it? That would defend against a partial set of statements from executing in the face of an error part way through.

athoscouto commented 1 year ago

Duplicate of https://github.com/libsql/libsql-shell-go/issues/75.