diff options
| author | Michael Paquier <michael@paquier.xyz> | 2022-04-18 13:41:40 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2022-04-18 13:41:40 +0900 |
| commit | 1a8b110539efe18803c1fa8aa452a2178dbad9a9 (patch) | |
| tree | aad5e0722e4a1d35ec0c0864fe5534ca0a09c213 | |
| parent | 676eeb6dd1655f7a3ee783b7ca0d645580630acc (diff) | |
| download | postgresql-1a8b110539efe18803c1fa8aa452a2178dbad9a9.tar.gz postgresql-1a8b110539efe18803c1fa8aa452a2178dbad9a9.zip | |
Fix second race condition in 002_archiving.pl with archive_cleanup_command
Checking the execution of archive_cleanup_command on a standby requires
a valid checkpoint coming from its primary, but the logic did not check
that the standby replayed up to the point of the checkpoint, causing the
test checking for the execution of archive_cleanup_command to fail.
This race was more visible in slow environments.
Issue introduced in 46dea24, so no backpatch is needed.
Author: Tom Lane
Discussion: https://postgr.es/m/4015413.1649454951@sss.pgh.pa.us
| -rw-r--r-- | src/test/recovery/t/002_archiving.pl | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/src/test/recovery/t/002_archiving.pl b/src/test/recovery/t/002_archiving.pl index 88ddc8fce79..01c52d8e7f7 100644 --- a/src/test/recovery/t/002_archiving.pl +++ b/src/test/recovery/t/002_archiving.pl @@ -46,13 +46,16 @@ $node_standby->start; # Create some content on primary $node_primary->safe_psql('postgres', "CREATE TABLE tab_int AS SELECT generate_series(1,1000) AS a"); -my $current_lsn = - $node_primary->safe_psql('postgres', "SELECT pg_current_wal_lsn();"); # Note the presence of this checkpoint for the archive_cleanup_command # check done below, before switching to a new segment. $node_primary->safe_psql('postgres', "CHECKPOINT"); +# Done after the checkpoint to ensure that it is replayed on the standby, +# for archive_cleanup_command. +my $current_lsn = + $node_primary->safe_psql('postgres', "SELECT pg_current_wal_lsn();"); + # Force archiving of WAL file to make it present on primary $node_primary->safe_psql('postgres', "SELECT pg_switch_wal()"); |