yiguolei commented on code in PR #33507:
URL: https://github.com/apache/doris/pull/33507#discussion_r1560349778
##########
fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java:
##########
@@ -601,7 +601,8 @@ public void execute(TUniqueId queryId) throws Exception {
if (e instanceof NereidsException &&
!context.getSessionVariable().enableFallbackToOriginalPlanner
&& !forceFallback) {
LOG.warn("Analyze failed. {}",
context.getQueryIdentifier(), e);
- throw ((NereidsException) e).getException();
+ context.getState().setError(e.getMessage());
Review Comment:
If the code like this:
statementexecutor.execute();
code1;
code2;
if you not throw exception,then code1 and code2 will run. BUT it will not
run in the past.
Any problems?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
