From 52f3a9d6a32c0c070a15486c3aecbc4405d2da88 Mon Sep 17 00:00:00 2001 From: Peter Eisentraut Date: Fri, 23 Mar 2018 17:18:22 -0400 Subject: [PATCH] Small refactoring Put the "atomic" argument of ExecuteDoStmt() and ExecuteCallStmt() into a variable instead of repeating the formula. --- src/backend/tcop/utility.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/backend/tcop/utility.c b/src/backend/tcop/utility.c index 8481fcca367..e144583bd1f 100644 --- a/src/backend/tcop/utility.c +++ b/src/backend/tcop/utility.c @@ -382,6 +382,7 @@ standard_ProcessUtility(PlannedStmt *pstmt, { Node *parsetree = pstmt->utilityStmt; bool isTopLevel = (context == PROCESS_UTILITY_TOPLEVEL); + bool isAtomicContext = (context != PROCESS_UTILITY_TOPLEVEL || IsTransactionBlock()); ParseState *pstate; check_xact_readonly(parsetree); @@ -514,8 +515,7 @@ standard_ProcessUtility(PlannedStmt *pstmt, break; case T_DoStmt: - ExecuteDoStmt((DoStmt *) parsetree, - (context != PROCESS_UTILITY_TOPLEVEL || IsTransactionBlock())); + ExecuteDoStmt((DoStmt *) parsetree, isAtomicContext); break; case T_CreateTableSpaceStmt: @@ -644,9 +644,7 @@ standard_ProcessUtility(PlannedStmt *pstmt, break; case T_CallStmt: - ExecuteCallStmt(castNode(CallStmt, parsetree), params, - (context != PROCESS_UTILITY_TOPLEVEL || IsTransactionBlock()), - dest); + ExecuteCallStmt(castNode(CallStmt, parsetree), params, isAtomicContext, dest); break; case T_ClusterStmt: -- 2.30.2