From b8830d063fb8f27079d0b75ff06517deaa16157d Mon Sep 17 00:00:00 2001 From: SaitTalhaNisanci Date: Fri, 10 Jul 2020 14:16:23 +0300 Subject: [PATCH] remove no-op check in TaskListRequires2PC (#4018) We already return true if replication model is REPLICATION_MODEL_2PC at the very beginning of the function, hence the check later is not used. --- src/backend/distributed/executor/adaptive_executor.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/backend/distributed/executor/adaptive_executor.c b/src/backend/distributed/executor/adaptive_executor.c index ad711720b..daaa75548 100644 --- a/src/backend/distributed/executor/adaptive_executor.c +++ b/src/backend/distributed/executor/adaptive_executor.c @@ -1481,8 +1481,7 @@ TaskListRequires2PC(List *taskList) if (task->taskType == DDL_TASK) { - if (MultiShardCommitProtocol == COMMIT_PROTOCOL_2PC || - task->replicationModel == REPLICATION_MODEL_2PC) + if (MultiShardCommitProtocol == COMMIT_PROTOCOL_2PC) { return true; }