somu-imply commented on code in PR #14886:
URL: https://github.com/apache/druid/pull/14886#discussion_r1317994404
##########
sql/src/main/java/org/apache/druid/sql/calcite/rule/DruidRules.java:
##########
@@ -115,13 +115,13 @@ public static List<RelOptRule> rules(PlannerContext
plannerContext)
retVal.add(DruidOuterQueryRule.WINDOW);
}
- if (plannerContext.featureAvailable(EngineFeature.UNNEST)) {
Review Comment:
Makes sense, I have added it back. I moved it from PlannerContext to always
return true to individual engines so that if needed we have the provision to
turn it off for MSQ but say keep it on for native
--
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]
