rdelaage
/
postgresql
mirror of https://git.postgresql.org/git/postgresql.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Make new replication slot test code less racy 

The new test code added in ead9e51e82 is racy -- it hinges on
shared-memory state, which changes before the WARNING message is logged.
Put it the other way around.

Backpatch to 13.

Author: Álvaro Herrera <alvherre@alvh.no-ip.org>
Discussion: https://postgr.es/m/202107161809.zclasccpfcg3@alvherre.pgsql
This commit is contained in:
Alvaro Herrera 2021-07-17 13:19:17 -04:00
parent 665c5855e5
commit 8589299e03
No known key found for this signature in database
GPG Key ID: 1C20ACB9D5C564AE
1 changed files with 15 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
src/test/recovery/t/019_replslot_limit.pl
View File

@ -177,18 +177,21 @@ ok( !find_in_log(
my $logstart = get_log_size($node_primary); my $logstart = get_log_size($node_primary);
advance_wal($node_primary, 7); advance_wal($node_primary, 7);
# This slot should be broken, wait for that to happen # wait until the WARNING is issued
$node_primary->poll_query_until( my $invalidated = 0;
'postgres', for (my $i = 0; $i < 10000; $i++)
qq[SELECT wal_status = 'lost' FROM pg_replication_slots {
WHERE slot_name = 'rep1']); if (find_in_log(
$node_primary,
# WARNING should be issued "invalidating slot \"rep1\" because its restart_lsn [0-9A-F/]+ exceeds max_slot_wal_keep_size",
ok( find_in_log( $logstart))
$node_primary, {
"invalidating slot \"rep1\" because its restart_lsn [0-9A-F/]+ exceeds max_slot_wal_keep_size", $invalidated = 1;
$logstart), last;
'check that the warning is logged'); }
usleep(100_000);
}
ok($invalidated, 'check that slot invalidation has been logged');
$result = $node_primary->safe_psql( $result = $node_primary->safe_psql(
'postgres', 'postgres',