external
/
citus
mirror of https://github.com/citusdata/citus.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix a flaky behaviour in shared_connection_stats 

With the previous query, we were not pushing down the pg_sleep hence the
number of connections to a worker could be different from run to run.
pull/5083/head
Sait Talha Nisanci 2021-01-28 15:08:16 +03:00 committed by SaitTalhaNisanci
parent 4aff833ca8
commit 3584fa11b0
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/test/regress/expected/shared_connection_stats.out
View File

@ -226,10 +226,11 @@ COMMIT;
-- now, some of the optional connections would be skipped, -- now, some of the optional connections would be skipped,
-- and only 5 connections are used per node -- and only 5 connections are used per node
BEGIN; BEGIN;
SELECT count(*), pg_sleep(0.1) FROM test; SET LOCAL citus.max_adaptive_executor_pool_size TO 16;
count | pg_sleep with cte_1 as (select pg_sleep(0.1) is null, a from test) SELECT a from cte_1 ORDER By 1 LIMIT 1;
a
--------------------------------------------------------------------- ---------------------------------------------------------------------
101 | 0
(1 row) (1 row)
SELECT SELECT

3
src/test/regress/sql/shared_connection_stats.sql
View File

@ -148,7 +148,8 @@ COMMIT;
-- now, some of the optional connections would be skipped, -- now, some of the optional connections would be skipped,
-- and only 5 connections are used per node -- and only 5 connections are used per node
BEGIN; BEGIN;
SELECT count(*), pg_sleep(0.1) FROM test; SET LOCAL citus.max_adaptive_executor_pool_size TO 16;
with cte_1 as (select pg_sleep(0.1) is null, a from test) SELECT a from cte_1 ORDER By 1 LIMIT 1;
SELECT SELECT
connection_count_to_node connection_count_to_node
FROM FROM