Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Reduce wal_retrieve_retry_interval in applicable TAP tests.
authorTom Lane <tgl@sss.pgh.pa.us>
Mon, 26 Jun 2017 23:01:26 +0000 (19:01 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Mon, 26 Jun 2017 23:01:26 +0000 (19:01 -0400)
By default, wal_retrieve_retry_interval is five seconds, which is far
more than is needed in any of our TAP tests, leaving the test cases
just twiddling their thumbs for significant stretches.  Moreover,
because it's so large, we get basically no testing of the retry-before-
master-is-ready code path.  Hence, make PostgresNode::init set up
wal_retrieve_retry_interval = '500ms' as part of its customization of
test clusters' postgresql.conf.  This shaves quite a few seconds off
the runtime of the recovery TAP tests.

Back-patch into 9.6.  We have wal_retrieve_retry_interval in 9.5,
but the test infrastructure isn't there.

Discussion: https://postgr.es/m/31624.1498500416@sss.pgh.pa.us

src/test/perl/PostgresNode.pm

index 65c4a8804520c784df32266b40d439436e4b1e43..711d0deae8f87a1f11f073b93b1413c3736c70e2 100644 (file)
@@ -413,6 +413,7 @@ sub init
    print $conf "fsync = off\n";
    print $conf "restart_after_crash = off\n";
    print $conf "log_statement = all\n";
+   print $conf "wal_retrieve_retry_interval = '500ms'\n";
    print $conf "port = $port\n";
 
    if ($params{allows_streaming})