Gleipnir/nidus-sync
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Properly close rows on empty results

Browse source 
I we don't do this we get "conn busy" errors.
This commit is contained in:
Eli Ribble 2026-05-08 22:21:27 +00:00
parent f2585c569c
commit 735a9dc1d2
No known key found for this signature in database
1 changed files with 12 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

16
db/connection.go
View file

@ -44,14 +44,22 @@ func ExecuteNone(ctx context.Context, stmt postgres.Statement) error {
func ExecuteNoneTx(ctx context.Context, txn Ex, stmt postgres.Statement) error { func ExecuteNoneTx(ctx context.Context, txn Ex, stmt postgres.Statement) error {
query, args := stmt.Sql() query, args := stmt.Sql()
_, err := txn.Query(ctx, query, args...) r, err := txn.Query(ctx, query, args...)
return err if err != nil {
return fmt.Errorf("query: %w", err)
}
r.Close()
return nil
} }
func ExecuteNoneTxBob(ctx context.Context, txn bob.Tx, stmt postgres.Statement) error { func ExecuteNoneTxBob(ctx context.Context, txn bob.Tx, stmt postgres.Statement) error {
query, args := stmt.Sql() query, args := stmt.Sql()
_, err := txn.QueryContext(ctx, query, args...) r, err := txn.QueryContext(ctx, query, args...)
return err if err != nil {
return fmt.Errorf("query: %w", err)
}
r.Close()
return nil
} }
func ExecuteOne[T any](ctx context.Context, stmt postgres.Statement) (T, error) { func ExecuteOne[T any](ctx context.Context, stmt postgres.Statement) (T, error) {
query, args := stmt.Sql() query, args := stmt.Sql()