database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

[opt](Nereids)when both Nereids and old parsers report errors, prompt error messages for the Nereids (#28580)

Browse Source
This commit is contained in:
zhangdong
2023-12-19 12:04:08 +08:00
committed by GitHub
parent 85317122cf
commit 87d36d0ffa
7 changed files with 14 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
fe/fe-core/src/main/java/org/apache/doris/qe/ConnectProcessor.java
View File

@ -182,7 +182,7 @@ public abstract class ConnectProcessor {
.setSqlHash(ctx.getSqlHash());
List<StatementBase> stmts = null;
Exception nereidsParseException = null;
// Nereids do not support prepare and execute now, so forbid prepare command, only process query command
if (mysqlCommand == MysqlCommand.COM_QUERY && ctx.getSessionVariable().isEnableNereidsPlanner()) {
try {
@ -195,6 +195,7 @@ public abstract class ConnectProcessor {
// TODO: We should catch all exception here until we support all query syntax.
LOG.debug("Nereids parse sql failed. Reason: {}. Statement: \"{}\".",
e.getMessage(), convertedStmt);
nereidsParseException = e;
}
}
@ -203,6 +204,12 @@ public abstract class ConnectProcessor {
try {
stmts = parse(convertedStmt);
} catch (Throwable throwable) {
// if NereidsParser and oldParser both failed,
// prove is a new feature implemented only on the nereids,
// so an error message for the new nereids is thrown
if (nereidsParseException != null) {
throwable = nereidsParseException;
}
// Parse sql failed, audit it and return
handleQueryException(throwable, convertedStmt, null, null);
return;