pgsql: Improve wait logic in TAP tests for streaming replication.

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: pgsql-committers(at)postgresql(dot)org
Subject: pgsql: Improve wait logic in TAP tests for streaming replication.
Date: 2017-06-26 18:39:59
Message-ID: E1dPYvX-0005UG-NZ@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Improve wait logic in TAP tests for streaming replication.

Remove hard-wired sleep(2) delays in 001_stream_rep.pl in favor of using
poll_query_until to check for the desired state to appear. In addition,
add such a wait before the last test in the script, as it's possible
to demonstrate failures there after upcoming improvements in pg_ctl.

(We might end up adding polling before each of the get_slot_xmins calls in
this script, but I feel no great need to do that until shown necessary.)

In passing, clarify the description strings for some of the test cases.

Michael Paquier and Craig Ringer, pursuant to a complaint from me

Discussion: https://postgr.es/m/8962.1498425057@sss.pgh.pa.us

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/5c77690f6f419c504b7d8248db30c2280217e72e

Modified Files
--------------
src/test/recovery/t/001_stream_rep.pl | 65 ++++++++++++++++++++++++-----------
1 file changed, 45 insertions(+), 20 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2017-06-26 19:13:29 pgsql: Reduce pg_ctl's reaction time when waiting for postmaster start/
Previous Message Tom Lane 2017-06-26 14:43:26 pgsql: Avoid useless "x = ANY(ARRAY[])" test for empty partition list.