From 93f1d13d18c4eb92d50421a4c0ff1bf235466336 Mon Sep 17 00:00:00 2001 From: Mehmet Yilmaz Date: Mon, 24 Feb 2025 14:26:28 +0000 Subject: [PATCH] Refactor SQL syntax in issue 7891 tests for consistency and clarity --- src/test/regress/expected/issue_7891.out | 18 +++++++++--------- src/test/regress/sql/issue_7891.sql | 10 +++++----- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/src/test/regress/expected/issue_7891.out b/src/test/regress/expected/issue_7891.out index fc5a55d14..16e2fbe41 100644 --- a/src/test/regress/expected/issue_7891.out +++ b/src/test/regress/expected/issue_7891.out @@ -207,14 +207,14 @@ SELECT 't2_ref after UPDATE' AS label, * FROM t2_ref; (1 row) -- Creating an additional reference table t3_ref to confirm subquery logic -create table t3_ref(pkey int, c15 text); -select create_reference_table('t3_ref'); +CREATE TABLE t3_ref(pkey INT, c15 TEXT); +SELECT create_reference_table('t3_ref'); create_reference_table --------------------------------------------------------------------- (1 row) -insert into t3_ref values (99, 'Initial Data'); +INSERT INTO t3_ref VALUES (99, 'Initial Data'); UPDATE t2_ref SET c15 = '2088-08-08 00:00:00'::timestamp WHERE EXISTS ( SELECT 1 FROM t3_ref); SELECT 't2_ref after UPDATE' AS label, * FROM t2_ref; label | vkey | pkey | c15 @@ -222,10 +222,10 @@ SELECT 't2_ref after UPDATE' AS label, * FROM t2_ref; t2_ref after UPDATE | 14 | 24000 | Sun Aug 08 00:00:00 2088 (1 row) -SELECT citus_remove_node('localhost', :worker_2_port); - citus_remove_node +SELECT 1 FROM citus_remove_node('localhost', :worker_2_port); + ?column? --------------------------------------------------------------------- - + 1 (1 row) SELECT 't2_ref after UPDATE - without worker 2' AS label, * FROM t2_ref; @@ -240,10 +240,10 @@ SELECT 1 FROM citus_add_node('localhost', :worker_2_port); 1 (1 row) -SELECT citus_remove_node('localhost', :worker_1_port); - citus_remove_node +SELECT 1 FROM citus_remove_node('localhost', :worker_1_port); + ?column? --------------------------------------------------------------------- - + 1 (1 row) SELECT 't2_ref after UPDATE - without worker 1' AS label, * FROM t2_ref; diff --git a/src/test/regress/sql/issue_7891.sql b/src/test/regress/sql/issue_7891.sql index 554cf0298..84881543d 100644 --- a/src/test/regress/sql/issue_7891.sql +++ b/src/test/regress/sql/issue_7891.sql @@ -165,20 +165,20 @@ UPDATE t2_ref SELECT 't2_ref after UPDATE' AS label, * FROM t2_ref; -- Creating an additional reference table t3_ref to confirm subquery logic -create table t3_ref(pkey int, c15 text); -select create_reference_table('t3_ref'); -insert into t3_ref values (99, 'Initial Data'); +CREATE TABLE t3_ref(pkey INT, c15 TEXT); +SELECT create_reference_table('t3_ref'); +INSERT INTO t3_ref VALUES (99, 'Initial Data'); UPDATE t2_ref SET c15 = '2088-08-08 00:00:00'::timestamp WHERE EXISTS ( SELECT 1 FROM t3_ref); SELECT 't2_ref after UPDATE' AS label, * FROM t2_ref; -SELECT citus_remove_node('localhost', :worker_2_port); +SELECT 1 FROM citus_remove_node('localhost', :worker_2_port); SELECT 't2_ref after UPDATE - without worker 2' AS label, * FROM t2_ref; SELECT 1 FROM citus_add_node('localhost', :worker_2_port); -SELECT citus_remove_node('localhost', :worker_1_port); +SELECT 1 FROM citus_remove_node('localhost', :worker_1_port); SELECT 't2_ref after UPDATE - without worker 1' AS label, * FROM t2_ref;