sohardforaname commented on code in PR #11035:
URL: https://github.com/apache/doris/pull/11035#discussion_r930773125


##########
fe/fe-core/src/main/java/org/apache/doris/nereids/NereidsPlanner.java:
##########
@@ -104,13 +105,18 @@ public PhysicalPlan plan(LogicalPlan plan, 
PhysicalProperties outputProperties,
                 // cascades style optimize phase.
                 .setJobContext(outputProperties);
 
+        finalizeAnalyze();
         rewrite();
         optimize();
 
         // Get plan directly. Just for SSB.
         return getRoot().extractPlan();
     }
 
+    private void finalizeAnalyze() {
+        new FinalizeAnalyzeJob(plannerContext).execute();

Review Comment:
   yes, because we should eliminate all the logical-subquery-alias nodes, or 
the rewrite phase will core.



-- 
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

Reply via email to