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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
