Put back explicit setting of replication values within TAP tests.

tglsfdc · tglsfdc · commit 496425304868 · 2020-10-01T10:59:20.000-04:00
Commit 151c0c5 neglected the possibility that a TEMP_CONFIG file would explicitly set max_wal_senders=0; as indeed buildfarm member thorntail does, so that it can test wal_level=minimal in other test suites. Hence, rather than assuming that max_wal_senders=10 will prevail if we say nothing, set it explicitly. Set max_replication_slots=10 explicitly too, just to be safe. Back-patch to v10, like the previous patch. Discussion: https://postgr.es/m/723911.1601417626@sss.pgh.pa.us
diff --git a/src/test/perl/PostgresNode.pm b/src/test/perl/PostgresNode.pm
@@ -469,12 +469,15 @@ sub init
 		{
 			print $conf "wal_level = replica\n";
 		}
-		print $conf "max_wal_size = 128MB\n";
+		print $conf "max_wal_senders = 10\n";
+		print $conf "max_replication_slots = 10\n";
 		print $conf "wal_log_hints = on\n";
 		print $conf "hot_standby = on\n";
 		# conservative settings to ensure we can run multiple postmasters:
 		print $conf "shared_buffers = 1MB\n";
 		print $conf "max_connections = 10\n";
+		# limit disk space consumption, too:
+		print $conf "max_wal_size = 128MB\n";
 	}
 	else
 	{

Original file line number	Diff line number	Diff line change
`@@ -469,12 +469,15 @@ sub init`
`469`	`469`	`{`
`470`	`470`	`print $conf "wal_level = replica\n";`
`471`	`471`	`}`
`472`		`- print $conf "max_wal_size = 128MB\n";`
	`472`	`+ print $conf "max_wal_senders = 10\n";`
	`473`	`+ print $conf "max_replication_slots = 10\n";`
`473`	`474`	`print $conf "wal_log_hints = on\n";`
`474`	`475`	`print $conf "hot_standby = on\n";`
`475`	`476`	`# conservative settings to ensure we can run multiple postmasters:`
`476`	`477`	`print $conf "shared_buffers = 1MB\n";`
`477`	`478`	`print $conf "max_connections = 10\n";`
	`479`	`+ # limit disk space consumption, too:`
	`480`	`+ print $conf "max_wal_size = 128MB\n";`
`478`	`481`	`}`
`479`	`482`	`else`
`480`	`483`	`{`