From d977b376bbcb94a12b292c8fb286b5eb0cb9b72e Mon Sep 17 00:00:00 2001 From: Denis Erokhin Date: Tue, 23 Jan 2024 12:10:35 +0300 Subject: [PATCH] Fix code formatting and change test result --- src/backend/distributed/executor/executor_util_tasks.c | 2 +- src/test/regress/expected/propagate_set_commands.out | 4 ++-- src/test/regress/sql/propagate_set_commands.sql | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/backend/distributed/executor/executor_util_tasks.c b/src/backend/distributed/executor/executor_util_tasks.c index 60ca84f14..863f3d8f9 100644 --- a/src/backend/distributed/executor/executor_util_tasks.c +++ b/src/backend/distributed/executor/executor_util_tasks.c @@ -81,7 +81,7 @@ TaskListRequiresRollback(List *taskList) if (ReadOnlyTask(task->taskType)) { return (SelectOpensTransactionBlock && IsTransactionBlock()) || - IsolationUsesXactSnapshot(); + IsolationUsesXactSnapshot(); } if (IsMultiStatementTransaction()) diff --git a/src/test/regress/expected/propagate_set_commands.out b/src/test/regress/expected/propagate_set_commands.out index fbfb370c1..3a23ad418 100644 --- a/src/test/regress/expected/propagate_set_commands.out +++ b/src/test/regress/expected/propagate_set_commands.out @@ -149,11 +149,11 @@ SELECT current_setting('transaction_isolation') FROM test WHERE id = 1; (1 row) END; --- SET is not propagated and plain SELECT does not use transaction blocks +-- SET is propagated due plain SELECT uses transaction blocks in repeatable read level SELECT DISTINCT current_setting('transaction_isolation') FROM test; current_setting --------------------------------------------------------------------- - read committed + repeatable read (1 row) -- the CTE will trigger transaction blocks diff --git a/src/test/regress/sql/propagate_set_commands.sql b/src/test/regress/sql/propagate_set_commands.sql index e30339d06..ce0c9b282 100644 --- a/src/test/regress/sql/propagate_set_commands.sql +++ b/src/test/regress/sql/propagate_set_commands.sql @@ -84,7 +84,7 @@ BEGIN; SELECT current_setting('transaction_isolation') FROM test WHERE id = 1; END; --- SET is not propagated and plain SELECT does not use transaction blocks +-- SET is propagated due plain SELECT uses transaction blocks in repeatable read level SELECT DISTINCT current_setting('transaction_isolation') FROM test; -- the CTE will trigger transaction blocks