Eschew "RESET statement_timeout" in tests.
authorNoah Misch
Tue, 20 Oct 2015 04:37:22 +0000 (00:37 -0400)
committerNoah Misch
Tue, 20 Oct 2015 04:37:45 +0000 (00:37 -0400)
commit0c7390d6240693eae1e357dc119b3a26c2836bb0
tree0418caa16af477198eb48f54b26db51b2117d2af
parente69d4756ef377f388c88ed8d4faad736751197cc
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.
src/test/regress/expected/plpgsql.out
src/test/regress/expected/prepared_xacts.out
src/test/regress/expected/prepared_xacts_1.out
src/test/regress/sql/plpgsql.sql
src/test/regress/sql/prepared_xacts.sql