]> granicus.if.org Git - postgresql/commit
Fix yet another race condition in recovery/t/001_stream_rep.pl.
authorTom Lane <tgl@sss.pgh.pa.us>
Tue, 8 Aug 2017 22:03:30 +0000 (18:03 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Tue, 8 Aug 2017 22:03:30 +0000 (18:03 -0400)
commit4576a69354fa2efc1bafa50df1c104c1a80c64e5
tree0a640e537e998f427706d4eb80eadd23cdc4e518
parentb2c95a3798ff39fc24d71b6655ddfe0e4cb3f378
Fix yet another race condition in recovery/t/001_stream_rep.pl.

In commit 5c77690f6, we added polling in front of most of the
get_slot_xmins calls in 001_stream_rep.pl, but today's results from
buildfarm member nightjar show that at least one more poll loop
is needed.

Proactively add a poll loop before the next-to-last get_slot_xmins call
as well.  It may be that there is no race condition there because the
standby_2 server is shut down at that point, but I'm quite tired of
fighting with this test script.  The empirical evidence that it's safe,
from the buildfarm, is no stronger than the evidence for the other
call that nightjar just proved unsafe.

The only remaining get_slot_xmins calls without wait_slot_xmins
protection are the first two, which should be OK since nothing has
happened at that point.  It's tempting to ignore that special case
and merge get_slot_xmins and wait_slot_xmins into a single function.
I didn't go that far though.

Discussion: https://postgr.es/m/18436.1502228036@sss.pgh.pa.us
src/test/recovery/t/001_stream_rep.pl