Use PG_TEST_TIMEOUT_DEFAULT in 019_replslot_limit.pl.
authorNoah Misch <[email protected]>
Sun, 2 Apr 2023 16:31:09 +0000 (09:31 -0700)
committerNoah Misch <[email protected]>
Sun, 2 Apr 2023 16:31:09 +0000 (09:31 -0700)
Oversight in f2698ea02ca8a56f38935d2b300ac54936712558, which introduced
the variable.  This lowers some 1000s timeouts to the configurable
default of 180s, due to a lack of evidence for needing the longer
timeout.  The alternative was 6*PG_TEST_TIMEOUT_DEFAULT, which we can
adopt if the need arises.  Given the lack of observed trouble with these
timeouts, no back-patch.

src/test/recovery/t/019_replslot_limit.pl

index 23f67093f632b70bb4b714ca5e93d55d1200eb0b..cb047bf77d94e6c41ff99f66d0c70804cb9b91cb 100644 (file)
@@ -182,7 +182,7 @@ $node_primary->safe_psql('postgres',
    'ALTER SYSTEM RESET max_wal_size; SELECT pg_reload_conf()');
 $node_primary->safe_psql('postgres', "CHECKPOINT;");
 my $invalidated = 0;
-for (my $i = 0; $i < 10000; $i++)
+for (my $i = 0; $i < 10 * $PostgreSQL::Test::Utils::timeout_default; $i++)
 {
    if (find_in_log(
            $node_primary, 'invalidating obsolete replication slot "rep1"',
@@ -205,7 +205,7 @@ is($result, "rep1|f|t|lost|",
 
 # Wait until current checkpoint ends
 my $checkpoint_ended = 0;
-for (my $i = 0; $i < 10000; $i++)
+for (my $i = 0; $i < 10 * $PostgreSQL::Test::Utils::timeout_default; $i++)
 {
    if (find_in_log($node_primary, "checkpoint complete: ", $logstart))
    {
@@ -235,7 +235,7 @@ $logstart = get_log_size($node_standby);
 $node_standby->start;
 
 my $failed = 0;
-for (my $i = 0; $i < 10000; $i++)
+for (my $i = 0; $i < 10 * $PostgreSQL::Test::Utils::timeout_default; $i++)
 {
    if (find_in_log(
            $node_standby,
@@ -353,8 +353,7 @@ while (1)
        stderr => \$stderr);
    diag $stdout, $stderr;
 
-   # unlikely that the problem would resolve after 15s, so give up at point
-   if ($i++ == 150)
+   if ($i++ == 10 * $PostgreSQL::Test::Utils::timeout_default)
    {
        # An immediate shutdown may hide evidence of a locking bug. If
        # retrying didn't resolve the issue, shut down in fast mode.