eldenmoon commented on code in PR #33403:
URL: https://github.com/apache/doris/pull/33403#discussion_r1577219222
##########
fe/fe-core/src/main/java/org/apache/doris/analysis/PrepareStmt.java:
##########
@@ -163,13 +164,15 @@ public void analyze(Analyzer analyzer) throws
UserException {
if (inner instanceof SelectStmt) {
// Use tmpAnalyzer since selectStmt will be reAnalyzed
Analyzer tmpAnalyzer = new Analyzer(context.getEnv(), context);
+ tmpAnalyzer.setPrepareStmt(true);
SelectStmt selectStmt = (SelectStmt) inner;
inner.analyze(tmpAnalyzer);
- if (!selectStmt.checkAndSetPointQuery()) {
- throw new UserException("Only support prepare SelectStmt point
query now");
+ isPointQuery = selectStmt.checkAndSetPointQuery();
+ if (isPointQuery) {
Review Comment:
statment should be these types `NONE, FULL_PREPARED, STATEMENT`, STATEMENT
type should suite. no need to `setPrepareStmt`, use type directly
--
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
