我需要显式回滚事务吗？

Question

我想知道 Go 如何处理失败的数据库事务。我的代码如下：

func assert(e interface{}) {
    if e != nil {
        panic(e)
    }
}

//the caller will handle panics
func SomeDBOperation(db *sql.DB) {
    tx, err := db.Begin()
    assert(err)
    defer func() {
        if e := recover(); e != nil {
            tx.Rollback()
            panic(e)
        }
        assert(tx.Commit())
    }()
    // some code that can possibly panic...
}

我可以像这样简化错误检查吗：

func SomeDBOperation(db *sql.DB) {
    tx, err := db.Begin()
    assert(err)
    defer func() { assert(tx.Commit()) }()
    // some code that can possibly panic...
}

顺便说一句，我正在使用 SQLite，如果有任何答案是特定于 db 的，我也想知道 MySQL 的行为。

Answer 1

默认情况下，任何数据库错误都会自动取消并回滚事务。这就是交易的用途。所以严格来说，在出现数据库错误（即外键违规什么的）的情况下，不需要自己回滚事务。

但是，您应该始终在创建事务后立即推迟回滚。这样，如果有任何错误与数据库无关，事务将回滚并清理。在这种情况下，回滚已经中止的事务将是无操作的，因此无害。

这在代码中看起来是这样的：

func SomeDBOperation(db *sql.DB) error {
    txn, err := db.Begin()
    if err != nil {
        return fmt.Errorf("failed to start transaction: %w", err)
    }
    defer txn.Rollback() // nolint:errcheck
    /* ... whatever other logic and DB operations you care about */
    return nil
}

Answer 2

如果在执行任何查询时出现错误，则回滚 tx 很重要，否则它仍在运行并持有锁。查看this 帖子。