Eschew "RESET statement_timeout" in tests.
Noah Misch <noah@leadboat.com>
Eschew "RESET statement_timeout" in tests. Instead, use transaction abort. Given an unlucky bout of latency, the timeout would cancel the RESET itself. Buildfarm members gharial, lapwing, mereswine, shearwater, and sungazer witness that. Back-patch to 9.1 (all supported versions). The query_canceled test still could timeout before entering its subtransaction; for whatever reason, that has yet to happen on the buildfarm.
Files
| Path | Change | +/− |
|---|---|---|
| src/test/regress/expected/plpgsql.out | modified | +29 −14 |
| src/test/regress/expected/prepared_xacts_1.out | modified | +4 −2 |
| src/test/regress/expected/prepared_xacts.out | modified | +4 −2 |
| src/test/regress/sql/plpgsql.sql | modified | +24 −13 |
| src/test/regress/sql/prepared_xacts.sql | modified | +4 −2 |