summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Paquier2021-03-22 00:51:05 +0000
committerMichael Paquier2021-03-22 00:51:05 +0000
commit992d353a190c551db39bcab2dec0ecf14fbc7a40 (patch)
tree647bc514059fcc16dde471f6601c397a4d2b2726
parent11e1577a576fec6307aa0bfcde7333e63f907fa7 (diff)
Fix new TAP test for 2PC transactions and PITRs on Windows
The test added by 595b9cb forgot that on Windows it is necessary to set up pg_hba.conf (see PostgresNode::set_replication_conf) with a specific entry or base backups fail. Any node that requires to support replication just needs to pass down allows_streaming at initialization. This updates the test to do so. Simplify things a bit while on it. Per buildfarm member fairywren. Any Windows hosts running this test would have failed, and I have reproduced the problem as well. Backpatch-through: 10
-rw-r--r--src/test/recovery/t/023_pitr_prepared_xact.pl5
1 files changed, 1 insertions, 4 deletions
diff --git a/src/test/recovery/t/023_pitr_prepared_xact.pl b/src/test/recovery/t/023_pitr_prepared_xact.pl
index 4edfe279bc0..533cd1a0fb3 100644
--- a/src/test/recovery/t/023_pitr_prepared_xact.pl
+++ b/src/test/recovery/t/023_pitr_prepared_xact.pl
@@ -8,11 +8,9 @@ use File::Compare;
# Initialize and start primary node with WAL archiving
my $node_primary = get_new_node('primary');
-$node_primary->init(has_archiving => 1);
+$node_primary->init(has_archiving => 1, allows_streaming => 1);
$node_primary->append_conf(
'postgresql.conf', qq{
-max_wal_senders = 10
-wal_level = 'replica'
max_prepared_transactions = 10});
$node_primary->start;
@@ -30,7 +28,6 @@ $node_pitr->init_from_backup(
has_restoring => 1);
$node_pitr->append_conf(
'postgresql.conf', qq{
-max_prepared_transactions = 10
recovery_target_name = 'rp'
recovery_target_action = 'promote'});