aboutsummaryrefslogtreecommitdiff
path: root/src/backend/tcop/postgres.c
diff options
context:
space:
mode:
authorTom Lane <tgl@sss.pgh.pa.us>2019-12-09 14:31:57 -0500
committerTom Lane <tgl@sss.pgh.pa.us>2019-12-09 14:31:57 -0500
commit99351a8b5a3882ee5d131bd37d03a67ab07b4fea (patch)
tree9a30925d473b935fb7766426caf6e82294b9ce29 /src/backend/tcop/postgres.c
parent2d0fdfaccec8b314895e026018874dcc5565b43e (diff)
downloadpostgresql-99351a8b5a3882ee5d131bd37d03a67ab07b4fea.tar.gz
postgresql-99351a8b5a3882ee5d131bd37d03a67ab07b4fea.zip
Improve isolationtester's timeout management.
isolationtester.c had a hard-wired limit of 3 minutes per test step. It now emerges that this isn't quite enough for some of the slowest buildfarm animals. This isn't the first time we've had to raise this limit (cf. 1db439ad4), so let's make it configurable. This patch raises the default to 5 minutes, and introduces an environment variable PGISOLATIONTIMEOUT that can be set if more time is needed, following the precedent of PGCTLTIMEOUT. Also, modify isolationtester so that when the timeout is hit, it explicitly reports having sent a cancel. This makes the regression failure log considerably more intelligible. (In the worst case, a timed-out test might actually be reported as "passing" without this extra output, so arguably this is a bug fix in itself.) In passing, update the README file, which had apparently not gotten touched when we added "make check" support here. Back-patch to 9.6; older versions don't have comparable timeout logic. Discussion: https://postgr.es/m/22964.1575842935@sss.pgh.pa.us
Diffstat (limited to 'src/backend/tcop/postgres.c')
0 files changed, 0 insertions, 0 deletions