somu-imply commented on code in PR #15241:
URL: https://github.com/apache/druid/pull/15241#discussion_r1373679396
##########
sql/src/main/java/org/apache/druid/sql/calcite/planner/CalciteRulesManager.java:
##########
@@ -237,7 +233,12 @@ public List<Program> programs(final PlannerContext
plannerContext)
boolean isDebug = plannerContext.queryContext().isDebug();
return ImmutableList.of(
- Programs.sequence(preProgram,
Programs.ofRules(druidConventionRuleSet(plannerContext))),
+ Programs.sequence(
Review Comment:
This is great to tell us how the plans look before the hep planner starts.
Probably put a comment here saying that these are printed only when the Debug
flag is set and developers should set that and run queries to get more info on
the plans.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
