db icon indicating copy to clipboard operation
db copied to clipboard

Roll back transaction on failed ExecContext

Open egoodhall opened this issue 3 years ago • 0 comments

Adds a call to sqlTx.Rollback() when an error is returned from compat.ExecContext in the SQLite adapter. This addresses #669 SQLite Adapter doesn't roll back transaction on error, which I ran into while writing a SQLite-backed web app.

This seems to have also addressed https://github.com/upper/db/issues/636 - I was able to remove the case that skipped the test for SQLite.

egoodhall avatar Sep 13 '22 03:09 egoodhall