This is an automated email from the ASF dual-hosted git repository.
morrysnow pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 46d3e90dc9e [chore](planner) remove useless legacy planner parse in
ConnectProcessor (#41248)
46d3e90dc9e is described below
commit 46d3e90dc9e459b2467afc5c61296b069e2dff86
Author: morrySnow <[email protected]>
AuthorDate: Thu Sep 26 14:58:15 2024 +0800
[chore](planner) remove useless legacy planner parse in ConnectProcessor
(#41248)
---
.../java/org/apache/doris/qe/ConnectProcessor.java | 35 ----------------------
1 file changed, 35 deletions(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/qe/ConnectProcessor.java
b/fe/fe-core/src/main/java/org/apache/doris/qe/ConnectProcessor.java
index 3a1e95ad6d3..42724eebbe1 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/qe/ConnectProcessor.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/qe/ConnectProcessor.java
@@ -22,8 +22,6 @@ import org.apache.doris.analysis.InsertStmt;
import org.apache.doris.analysis.KillStmt;
import org.apache.doris.analysis.LiteralExpr;
import org.apache.doris.analysis.QueryStmt;
-import org.apache.doris.analysis.SqlParser;
-import org.apache.doris.analysis.SqlScanner;
import org.apache.doris.analysis.StatementBase;
import org.apache.doris.analysis.UserIdentity;
import org.apache.doris.catalog.Column;
@@ -43,7 +41,6 @@ import org.apache.doris.common.NotImplementedException;
import org.apache.doris.common.Pair;
import org.apache.doris.common.UserException;
import org.apache.doris.common.util.DebugUtil;
-import org.apache.doris.common.util.SqlParserUtils;
import org.apache.doris.common.util.SqlUtils;
import org.apache.doris.common.util.Util;
import org.apache.doris.datasource.CatalogIf;
@@ -87,7 +84,6 @@ import org.apache.logging.log4j.Logger;
import org.apache.thrift.TException;
import java.io.IOException;
-import java.io.StringReader;
import java.nio.ByteBuffer;
import java.util.ArrayList;
import java.util.List;
@@ -492,37 +488,6 @@ public abstract class ConnectProcessor {
auditAfterExec(origStmt, parsedStmt, statistics, true);
}
- // analyze the origin stmt and return multi-statements
- protected List<StatementBase> parse(String originStmt) throws
AnalysisException, DdlException {
- if (LOG.isDebugEnabled()) {
- LOG.debug("the originStmts are: {}", originStmt);
- }
- // Parse statement with parser generated by CUP&FLEX
- SqlScanner input = new SqlScanner(new StringReader(originStmt),
ctx.getSessionVariable().getSqlMode());
- SqlParser parser = new SqlParser(input);
- try {
- return SqlParserUtils.getMultiStmts(parser);
- } catch (Error e) {
- throw new AnalysisException("Please check your sql, we meet an
error when parsing.", e);
- } catch (AnalysisException | DdlException e) {
- String errorMessage = parser.getErrorMsg(originStmt);
- if (LOG.isDebugEnabled()) {
- LOG.debug("origin stmt: {}; Analyze error message: {}",
originStmt, parser.getErrorMsg(originStmt), e);
- }
- if (errorMessage == null) {
- throw e;
- } else {
- throw new AnalysisException(errorMessage, e);
- }
- } catch (ArrayStoreException e) {
- throw new AnalysisException("Sql parser can't convert the result
to array, please check your sql.", e);
- } catch (Exception e) {
- // TODO(lingbin): we catch 'Exception' to prevent unexpected error,
- // should be removed this try-catch clause future.
- throw new AnalysisException("Internal Error, maybe syntax error or
this is a bug: " + e.getMessage(), e);
- }
- }
-
// Get the column definitions of a table
@SuppressWarnings("rawtypes")
protected void handleFieldList(String tableName) throws
ConnectionException {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
