fix: not need to bring 'set autocommit = 0' before executing DDL

This commit is contained in:
wd2365151147
2022-01-14 15:24:41 +08:00
committed by Huqing Yan
parent c25605dcf0
commit e54cfdca8a
3 changed files with 3 additions and 3 deletions
@@ -135,7 +135,7 @@ public abstract class BaseDDLHandler implements ResponseHandler, ExecutableHandl
DDLTraceHelper.log(session.getShardingService(), d -> d.infoByNode(node.getName(), stage, DDLTraceHelper.Status.get_conn, "Get " + conn.toString()));
conn.getBackendService().setResponseHandler(this);
conn.getBackendService().setSession(session);
conn.getBackendService().execute(node, session.getShardingService(), session.getShardingService().isAutocommit() && !session.getShardingService().isTxStart());
conn.getBackendService().execute(node, session.getShardingService(), sessionAutocommit && !session.getShardingService().isTxStart());
}
@Override
@@ -25,6 +25,6 @@ public class MultiNodeDDLExecuteHandler extends BaseDDLHandler {
}
DDLTraceHelper.log(session.getShardingService(), d -> d.infoByNode(node.getName(), stage, DDLTraceHelper.Status.get_conn, "Get " + conn.toString()));
conn.getBackendService().setSession(session);
conn.getBackendService().executeMultiNode(node, session.getShardingService(), sessionAutocommit && !session.getShardingService().isTxStart() && !node.isModifySQL());
conn.getBackendService().executeMultiNode(node, session.getShardingService(), sessionAutocommit && !session.getShardingService().isTxStart());
}
}
@@ -44,7 +44,7 @@ public class MultiNodeDdlPrepareHandler extends BaseDDLHandler {
conn.getBackendService().setSession(session);
conn.getBackendService().setTesting(true);
conn.getBackendService().setComplexQuery(true);
conn.getBackendService().execute(node, session.getShardingService(), session.getShardingService().isAutocommit() && !session.getShardingService().isTxStart());
conn.getBackendService().execute(node, session.getShardingService(), sessionAutocommit && !session.getShardingService().isTxStart());
}
@Override