Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 8cca660

Browse files
committed
Disable use of archiving in 009_twophase.pl
This partially reverts 68cb5af, as using archiving to enforce the rename of the last partial segment of the old timeline at promotion to use .partial as suffix is impacting the tests when it does switchovers. As showed by the logs gathered by the CI in the tests that failed, a new standby may fail to find the WAL segment it needs to follow a promoted instance with its timeline jump, as it got renamed to .partial. This problem would manifest as a run timeout with 009_twophase.pl, as the new standby repeatedly requests a segment from the promoted primary that it would not find. Reported-by: Nathan Bossart Discussion: https://postgr.es/m/20230621043345.GA787473@nathanxps13 Backpatch-through: 13
1 parent a734caa commit 8cca660

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

src/test/recovery/t/009_twophase.pl

+1-3
Original file line numberDiff line numberDiff line change
@@ -32,9 +32,7 @@ sub configure_and_reload
3232

3333
# Setup london node
3434
my $node_london = PostgreSQL::Test::Cluster->new("london");
35-
# Archiving is used to provide coverage with the creation of .partial segments
36-
# done at the end of recovery and the recovery of two-phase transactions.
37-
$node_london->init(allows_streaming => 1, has_archiving => 1);
35+
$node_london->init(allows_streaming => 1);
3836
$node_london->append_conf(
3937
'postgresql.conf', qq(
4038
max_prepared_transactions = 10

0 commit comments

Comments
 (0)