cybertec-postgresql / pgwatch / 20032611416

75%

Build Type

push

github

Committed by web-flow

Commit Message

[*] set lock_timeout via connection RuntimeParams instead of per-query transaction (#1067)

* [*] set lock_timeout via connection RuntimeParams instead of per-query transaction

Reduces metric queries from 4 roundtrips (BEGIN/SET/query/COMMIT) to 1 by
setting lock_timeout at connection level via pgx RuntimeParams.

- Set lock_timeout in RuntimeParams for PostgreSQL sources on connect
- Simplify QueryMeasurements by removing transaction wrapper
- Set to "0" to disable lock_timeout

Ref: https://brandur.org/fragments/postgres-parameters

* [*] update tests to match new connection-level lock_timeout

Tests no longer expect transaction wrapping (BEGIN/SET/COMMIT) since
lock_timeout is now set via RuntimeParams at connection time.

* remove proposed `ConnLockTimeout` option

User may control connection options through connection string, e.g.
`--source=postgresql://foo:baz@bar/db?lock_timeout=100`

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Pavlo Golub <pavlo.golub@gmail.com>

Run Details

2 of 2 new or added lines in 1 file covered. (100.0%)

1 existing line in 1 file now uncovered.

3766 of 5090 relevant lines covered (73.99%)

0.84 hits per line