From: Noah Misch <noah@leadboat.com>
Date: Tue, 14 Apr 2020 01:47:28 +0000 (-0700)
Subject: Add a wait_for_catchup() before immediate stop of a test master.
X-Git-Tag: REL_13_BETA1~250
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d60cfb6bf2b4812e97271e7b5ba3ad4713406b9d;p=thirdparty%2Fpostgresql.git

Add a wait_for_catchup() before immediate stop of a test master.

Per buildfarm member hoverfly, a slow walsender could make the test
fail.  Back-patch to v10, where the test was introduced.

Discussion: https://postgr.es/m/20200414013849.GA886648@rfd.leadboat.com
---

diff --git a/src/test/recovery/t/010_logical_decoding_timelines.pl b/src/test/recovery/t/010_logical_decoding_timelines.pl
index e582b200050..11f5595e2bf 100644
--- a/src/test/recovery/t/010_logical_decoding_timelines.pl
+++ b/src/test/recovery/t/010_logical_decoding_timelines.pl
@@ -131,6 +131,7 @@ cmp_ok(
 	'xmin on physical slot must not be lower than catalog_xmin');
 
 $node_master->safe_psql('postgres', 'CHECKPOINT');
+$node_master->wait_for_catchup($node_replica, 'write');
 
 # Boom, crash
 $node_master->stop('immediate');