From 824a1724013f9b2486fe4da1f402d6df66e4589e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?G=C3=BCrkan=20=C4=B0ndibay?= Date: Fri, 19 Apr 2024 11:01:53 +0300 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Onur Tirtir --- src/test/regress/citus_tests/run_test.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/test/regress/citus_tests/run_test.py b/src/test/regress/citus_tests/run_test.py index f22a097b0..45a52fe2c 100755 --- a/src/test/regress/citus_tests/run_test.py +++ b/src/test/regress/citus_tests/run_test.py @@ -422,9 +422,12 @@ def find_test_schedule_and_line(test_name, args): def test_dependencies(test_name, test_schedule, schedule_line, args): if test_name in DEPS: - # since enable_ddl_propagation is a must to execute tests + # Since enable_ddl_propagation is a must to execute tests, # below block adds enable_ddl_propagation as a dependency - # as the first element of extra tests if schedule is not configured + # as the first element of extra tests if schedule is not + # configured. We don't do so if the schedule is confugured + # because all base schedules include enable_ddl_propagation + # anyway. test_deps = DEPS[test_name] ddl_propagation_test = ( "isolation_enable_ddl_propagation"