From eaf5321c352478266cebe2aa50ea4c34a8fdd2c7 Mon Sep 17 00:00:00 2001 From: Amit Kapila Date: Mon, 2 Aug 2021 08:29:26 +0530 Subject: [PATCH] Fix test failure in 021_twophase.pl. The test is expecting two prepared transactions corresponding to two subscriptions but it waits to catch up for just one subscription. Fix it by allowing to wait for both subscriptions. Reported-by: Michael Paquier, as per buildfarm Author: Ajin Cherian Reviewed-By: Amit Kapila, Vignesh C, Peter Smith Discussion: https://postgr.es/m/CAA4eK1+_0iNQ8Z=KVTjmmAqNX-hyv+1+fnZ-Yx8CVP=uAcekqw@mail.gmail.com --- src/test/subscription/t/021_twophase.pl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/test/subscription/t/021_twophase.pl b/src/test/subscription/t/021_twophase.pl index 903a771fe3..19f096295f 100644 --- a/src/test/subscription/t/021_twophase.pl +++ b/src/test/subscription/t/021_twophase.pl @@ -316,7 +316,9 @@ $node_publisher->safe_psql('postgres', " INSERT INTO tab_copy VALUES (99); PREPARE TRANSACTION 'mygid';"); +# Wait for both subscribers to catchup $node_publisher->wait_for_catchup($appname_copy); +$node_publisher->wait_for_catchup($appname); # Check that the transaction has been prepared on the subscriber, there will be 2 # prepared transactions for the 2 subscriptions.