From 48caca40848ccad5c8447d4172044f949f543607 Mon Sep 17 00:00:00 2001 From: Marco Slot Date: Mon, 30 Nov 2020 09:49:10 +0100 Subject: [PATCH] Improve regression test settings --- src/test/regress/expected/shared_connection_stats.out | 2 ++ src/test/regress/pg_regress_multi.pl | 4 ++++ src/test/regress/sql/shared_connection_stats.sql | 2 ++ 3 files changed, 8 insertions(+) diff --git a/src/test/regress/expected/shared_connection_stats.out b/src/test/regress/expected/shared_connection_stats.out index 72b654488..49cc8ef1f 100644 --- a/src/test/regress/expected/shared_connection_stats.out +++ b/src/test/regress/expected/shared_connection_stats.out @@ -226,6 +226,7 @@ COMMIT; -- now, some of the optional connections would be skipped, -- and only 5 connections are used per node BEGIN; + SET LOCAL citus.max_adaptive_executor_pool_size TO 16; SELECT count(*), pg_sleep(0.1) FROM test; count | pg_sleep --------------------------------------------------------------------- @@ -476,6 +477,7 @@ BEGIN; -- when COPY is used with _max_query_parallelization -- it ignores the shared pool size SET LOCAL citus.force_max_query_parallelization TO ON; + SET LOCAL citus.max_adaptive_executor_pool_size TO 16; COPY test FROM PROGRAM 'seq 32'; SELECT connection_count_to_node diff --git a/src/test/regress/pg_regress_multi.pl b/src/test/regress/pg_regress_multi.pl index e7065bd29..2216f21e4 100755 --- a/src/test/regress/pg_regress_multi.pl +++ b/src/test/regress/pg_regress_multi.pl @@ -415,11 +415,15 @@ push(@pgOptions, "shared_preload_libraries='${sharedPreloadLibraries}'"); # Avoid parallelism to stabilize explain plans push(@pgOptions, "max_parallel_workers_per_gather=0"); +# Help with debugging +push(@pgOptions, "log_error_verbosity = 'verbose'"); + # Allow CREATE SUBSCRIPTION to work push(@pgOptions, "wal_level='logical'"); # Citus options set for the tests push(@pgOptions, "citus.shard_count=4"); +push(@pgOptions, "citus.max_adaptive_executor_pool_size=4"); push(@pgOptions, "citus.shard_max_size=1500kB"); push(@pgOptions, "citus.repartition_join_bucket_count_per_node=2"); push(@pgOptions, "citus.sort_returning='on'"); diff --git a/src/test/regress/sql/shared_connection_stats.sql b/src/test/regress/sql/shared_connection_stats.sql index 7488f2d88..bd11f55fe 100644 --- a/src/test/regress/sql/shared_connection_stats.sql +++ b/src/test/regress/sql/shared_connection_stats.sql @@ -148,6 +148,7 @@ COMMIT; -- now, some of the optional connections would be skipped, -- and only 5 connections are used per node BEGIN; + SET LOCAL citus.max_adaptive_executor_pool_size TO 16; SELECT count(*), pg_sleep(0.1) FROM test; SELECT connection_count_to_node @@ -325,6 +326,7 @@ BEGIN; -- when COPY is used with _max_query_parallelization -- it ignores the shared pool size SET LOCAL citus.force_max_query_parallelization TO ON; + SET LOCAL citus.max_adaptive_executor_pool_size TO 16; COPY test FROM PROGRAM 'seq 32'; SELECT