jackwener commented on code in PR #13679:
URL: https://github.com/apache/doris/pull/13679#discussion_r1005305821
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/exploration/join/JoinReorderContext.java:
##########
@@ -36,6 +38,7 @@ public class JoinReorderContext {
private boolean hasRightAssociate = false;
private boolean hasLeftAssociate = false;
+ boolean ordered = false;
Review Comment:
remove these changes in this file.
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/NereidsPlanner.java:
##########
@@ -129,6 +132,8 @@ public PhysicalPlan plan(LogicalPlan plan,
PhysicalProperties outputProperties)
deriveStats();
+ // We need to do join reorder before cascades and after deriving stats
+ joinReorder();
Review Comment:
Comment out, as it will affect the current operation
--
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
