28319702248
94%

Ran 28 Jun 2026 10:51AM UTC

Jobs 1

Files 43

Run time 1min

Badge

Embed ▾

Committed 28 Jun 2026 10:45AM UTC coverage: 93.824% (-0.4%) from 94.224%

Build # 28319702248

Build Type

push

github

Committed by

web-flow

Commit Message

fix(pgxv4, pgxv5): remove awaitDone to prevent concurrent pgx.Tx access (#167)

## Fix
Remove the awaitDone goroutine from the pgx v4 and pgx v5 drivers. It called Rollback(ctx) on context cancellation concurrently with the user's in-flight query on the same pgx.Tx. Since pgx.Tx is not safe for concurrent use (jackc/pgx#2332), this could panic with BUG: slow write timer already active (#139).

Commit/rollback are now driven solely by the caller. With manager.Manager it's transparent — closer runs Rollback after the function returns, so nothing touches pgx.Tx concurrently. The now-unused sync.Mutex is removed too.

## Trade-off
Standalone use (without the manager) no longer auto-rolls-back on cancel — the caller must call Rollback explicitly. Rollback still forwards ctx, so on a cancelled context it returns context.Canceled and is not idempotent (pgx.ErrTxClosed on the second call).

---------

Co-authored-by: Ilia Sergunin <ilia.sergunin@gmail.com>

Coverage Stats

1580 of 1684 relevant lines covered (93.82%)

8.71 hits per line

Coverage Regressions

Lines	Coverage	∆	File
3	92.68	-5.53%	drivers/pgxv4/v2/transaction.go
3	92.68	-7.32%	drivers/pgxv5/v2/transaction.go

Jobs

ID	Job ID	Ran	Files	Coverage
1	28319702248.1	28 Jun 2026 10:51AM UTC	43	93.82	GitHub Action Run

avito-tech / go-transaction-manager / 28319702248
94%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Coverage Regressions

Jobs

Source Files on build 28319702248

avito-tech / go-transaction-manager / 28319702248 94%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Coverage Regressions

Jobs

Source Files on build 28319702248

avito-tech / go-transaction-manager / 28319702248
94%

README BADGES
x