Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit151c0c5

Browse files
committed
Remove obsolete replication settings within TAP tests.
PostgresNode.pm set "max_wal_senders = 5" for replication testing,but this seems to be slightly too low for our current test suite.Slower buildfarm members frequently report "number of requested standbyconnections exceeds max_wal_senders" failures, due to old walsendersnot exiting instantaneously. Usually, the test does not fail overallbecause of automatic walreceiver restart, but sometimes the failurebecomes visible; and in any case such retries slow down the test.That value came in with commit89ac700, but was soon obsoleted byf6d6d29, which raised the built-in default from zero to 10; so thatPostgresNode.pm is actually setting it to less than the conservativebuilt-in default. That seems pretty pointless, so let's remove thespecial setting and let the default prevail, in hopes of makingthe TAP tests more robust.Likewise, the setting "max_replication_slots = 5" is obsolete andcan be removed.While here, reverse-engineer a comment about why we're choosingless-than-default values for some other settings.(Note: before v12, max_wal_senders counted against max_connectionsso that the latter setting also needs some fiddling with.)Back-patch to v10 where the subscription tests were added.It's likely that the older branches aren't pushing the boundariesof max_wal_senders, but I'm disinclined to spend time trying tofigure out exactly when it started to be a problem.Discussion:https://postgr.es/m/723911.1601417626@sss.pgh.pa.us
1 parent2b88864 commit151c0c5

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

‎src/test/perl/PostgresNode.pm

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -469,12 +469,11 @@ sub init
469469
{
470470
print$conf"wal_level = replica\n";
471471
}
472-
print$conf"max_wal_senders = 5\n";
473-
print$conf"max_replication_slots = 5\n";
474472
print$conf"max_wal_size = 128MB\n";
475-
print$conf"shared_buffers = 1MB\n";
476473
print$conf"wal_log_hints = on\n";
477474
print$conf"hot_standby = on\n";
475+
# conservative settings to ensure we can run multiple postmasters:
476+
print$conf"shared_buffers = 1MB\n";
478477
print$conf"max_connections = 10\n";
479478
}
480479
else

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp