summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAmit Kapila <akapila@postgresql.org>2021-08-02 08:29:26 +0530
committerAmit Kapila <akapila@postgresql.org>2021-08-02 08:31:48 +0530
commiteaf5321c352478266cebe2aa50ea4c34a8fdd2c7 (patch)
tree66408384411bbc216de0e2dac9ba40ef3704d1ce
parent0d1401931868e03342ed4689b5af2522eb40f750 (diff)
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
-rw-r--r--src/test/subscription/t/021_twophase.pl2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/test/subscription/t/021_twophase.pl b/src/test/subscription/t/021_twophase.pl
index 903a771fe35..19f096295f2 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.