diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2017-06-26 19:01:26 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2017-06-26 19:01:26 -0400 |
| commit | df31a9fc66f11e824872dc09d22f389cbd5bc803 (patch) | |
| tree | 76bb4abf423156b2d8f5c19ac988635fe020c5e3 | |
| parent | a4d1ce095bbebeefe055c74fafbb6ea736058e11 (diff) | |
| download | postgresql-df31a9fc66f11e824872dc09d22f389cbd5bc803.tar.gz postgresql-df31a9fc66f11e824872dc09d22f389cbd5bc803.zip | |
Reduce wal_retrieve_retry_interval in applicable TAP tests.
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
| -rw-r--r-- | src/test/perl/PostgresNode.pm | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/test/perl/PostgresNode.pm b/src/test/perl/PostgresNode.pm index 65c4a880452..711d0deae8f 100644 --- a/src/test/perl/PostgresNode.pm +++ b/src/test/perl/PostgresNode.pm @@ -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}) |