seawinde commented on code in PR #27231:
URL: https://github.com/apache/doris/pull/27231#discussion_r1403395840
##########
fe/fe-core/src/main/java/org/apache/doris/analysis/InlineViewRef.java:
##########
@@ -186,7 +189,13 @@ public void analyze(Analyzer analyzer) throws
AnalysisException, UserException {
}
if (view == null && !hasExplicitAlias()) {
Review Comment:
this is only for old optimizer, if support new optimizer, should also add
logical in
`org.apache.doris.nereids.parser.LogicalPlanBuilder#visitAliasedQuery`
maybe it's better to add alias in logicalPlanBuiler when inline view with
out alias rather than add the if logic in analyze
--
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
