From 50e6c505346cb79243bf69145162da897da6c193 Mon Sep 17 00:00:00 2001 From: Naisila Puka <37271756+naisila@users.noreply.github.com> Date: Fri, 9 Jun 2023 15:59:30 +0300 Subject: [PATCH] Remove flaky rebalance plan from test (#6990) Looks like sometimes shards are a slightly different size than we expect, 16k vs 8k, resulting in a different rebalance plan. --- .../expected/background_rebalance_parallel.out | 11 ----------- .../regress/sql/background_rebalance_parallel.sql | 4 ---- 2 files changed, 15 deletions(-) diff --git a/src/test/regress/expected/background_rebalance_parallel.out b/src/test/regress/expected/background_rebalance_parallel.out index dbdd963a9..cc3470de9 100644 --- a/src/test/regress/expected/background_rebalance_parallel.out +++ b/src/test/regress/expected/background_rebalance_parallel.out @@ -466,17 +466,6 @@ SELECT citus_rebalance_start AS job_id from citus_rebalance_start() \gset -- see dependent tasks to understand which tasks remain runnable because of -- citus.max_background_task_executors_per_node -- and which tasks are actually blocked from colocation group dependencies -SELECT (SELECT T.command FROM pg_dist_background_task T WHERE T.task_id = D.task_id), - (SELECT T.command depends_on_command FROM pg_dist_background_task T WHERE T.task_id = D.depends_on) -FROM pg_dist_background_task_depend D WHERE job_id in (:job_id) ORDER BY 1, 2 ASC; - command | depends_on_command ---------------------------------------------------------------------- - SELECT pg_catalog.citus_move_shard_placement(85674026,50,57,'auto') | SELECT pg_catalog.citus_move_shard_placement(85674025,50,56,'auto') - SELECT pg_catalog.citus_move_shard_placement(85674032,50,57,'auto') | SELECT pg_catalog.citus_move_shard_placement(85674031,50,56,'auto') - SELECT pg_catalog.citus_move_shard_placement(85674038,50,57,'auto') | SELECT pg_catalog.citus_move_shard_placement(85674037,50,56,'auto') - SELECT pg_catalog.citus_move_shard_placement(85674044,50,57,'auto') | SELECT pg_catalog.citus_move_shard_placement(85674043,50,56,'auto') -(4 rows) - SELECT task_id, depends_on FROM pg_dist_background_task_depend WHERE job_id in (:job_id) diff --git a/src/test/regress/sql/background_rebalance_parallel.sql b/src/test/regress/sql/background_rebalance_parallel.sql index 2eb952b67..8f8ed4d74 100644 --- a/src/test/regress/sql/background_rebalance_parallel.sql +++ b/src/test/regress/sql/background_rebalance_parallel.sql @@ -204,10 +204,6 @@ SELECT citus_rebalance_start AS job_id from citus_rebalance_start() \gset -- see dependent tasks to understand which tasks remain runnable because of -- citus.max_background_task_executors_per_node -- and which tasks are actually blocked from colocation group dependencies -SELECT (SELECT T.command FROM pg_dist_background_task T WHERE T.task_id = D.task_id), - (SELECT T.command depends_on_command FROM pg_dist_background_task T WHERE T.task_id = D.depends_on) -FROM pg_dist_background_task_depend D WHERE job_id in (:job_id) ORDER BY 1, 2 ASC; - SELECT task_id, depends_on FROM pg_dist_background_task_depend WHERE job_id in (:job_id)