From 30afff385dd292c0e057f837838f063ea16ebef8 Mon Sep 17 00:00:00 2001 From: Noah Misch Date: Mon, 13 Apr 2020 18:47:28 -0700 Subject: [PATCH] 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 --- src/test/recovery/t/010_logical_decoding_timelines.pl | 1 + 1 file changed, 1 insertion(+) diff --git a/src/test/recovery/t/010_logical_decoding_timelines.pl b/src/test/recovery/t/010_logical_decoding_timelines.pl index e582b20005..11f5595e2b 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');