yiguolei commented on code in PR #20048:
URL: https://github.com/apache/doris/pull/20048#discussion_r1205407013
##########
fe/fe-core/src/main/java/org/apache/doris/qe/StmtExecutor.java:
##########
@@ -885,6 +891,18 @@ public void analyze(TQueryOptions tQueryOptions) throws
UserException {
if (parsedStmt instanceof QueryStmt) {
queryStmt = (QueryStmt) parsedStmt;
queryStmt.getTables(analyzer, false, tableMap,
parentViewNameSet);
+
+ // queue query here
+ if (Config.enable_resource_group && Config.enable_query_queue)
{
Review Comment:
why in analyze method? Should put it into queue after analysis and before
send fragment to BE. So that if a query has 语法错误， then user could get analysis
error as soon as possible.
--
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
