diff options
| author | Michael Paquier <michael@paquier.xyz> | 2023-06-21 16:16:24 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2023-06-21 16:16:24 +0900 |
| commit | 2634926906ded28212abb110e3152337467535b3 (patch) | |
| tree | 973154faf541820a4eca26e13a39644c24a833b0 | |
| parent | 0b79042701ff845b9a580443c822d52566c1ec24 (diff) | |
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
| -rw-r--r-- | src/test/recovery/t/009_twophase.pl | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/test/recovery/t/009_twophase.pl b/src/test/recovery/t/009_twophase.pl index cf2f345269f..900d181788c 100644 --- a/src/test/recovery/t/009_twophase.pl +++ b/src/test/recovery/t/009_twophase.pl @@ -32,9 +32,7 @@ sub configure_and_reload # Setup london node my $node_london = get_new_node("london"); -# Archiving is used to provide coverage with the creation of .partial segments -# done at the end of recovery and the recovery of two-phase transactions. -$node_london->init(allows_streaming => 1, has_archiving => 1); +$node_london->init(allows_streaming => 1); $node_london->append_conf( 'postgresql.conf', qq( max_prepared_transactions = 10 |