You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by yi...@apache.org on 2024/01/29 09:33:36 UTC
(doris) branch branch-2.0 updated: [fix](query state) Print correct DML state (#30488)
This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-2.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-2.0 by this push:
new d5856bdea92 [fix](query state) Print correct DML state (#30488)
d5856bdea92 is described below
commit d5856bdea92da99a148c3653e8e38ce1f69e21f5
Author: zhiqiang <se...@163.com>
AuthorDate: Mon Jan 29 17:33:27 2024 +0800
[fix](query state) Print correct DML state (#30488)
---
fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java b/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java
index b764fbccc6d..bd385e1bfa6 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java
@@ -452,6 +452,11 @@ public class StmtExecutor {
LOG.debug("fall back to legacy planner on statement:\n{}", originStmt.originStmt);
parsedStmt = null;
planner = null;
+ // Attention: currently exception from nereids does not mean an Exception to user terminal
+ // unless user does not allow fallback to lagency planner. But state of query
+ // has already been set to Error in this case, it will have some side effect on profile result
+ // and audit log. So we need to reset state to OK if query cancel be processd by lagency.
+ context.getState().reset();
context.getState().setNereids(false);
executeByLegacy(queryId);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org