adarshsanjeev commented on code in PR #17541:
URL: https://github.com/apache/druid/pull/17541#discussion_r1883502919
##########
sql/src/main/java/org/apache/druid/sql/calcite/planner/CalcitePlanner.java:
##########
@@ -502,4 +510,41 @@ void from(CalcitePlanner planner)
+ " to " + this);
}
}
+
+ /** Define some tool members and methods for hints. */
+ private static class HintTools
+ {
+ static final HintStrategyTable HINT_STRATEGY_TABLE =
createHintStrategies();
+
+ /**
+ * Creates hint strategies.
+ *
+ * @return HintStrategyTable instance
+ */
+ private static HintStrategyTable createHintStrategies()
+ {
+ return HintStrategyTable.builder()
+ .hintStrategy("broadcast", HintPredicates.JOIN)
+ .hintStrategy("sort_merge", HintPredicates.JOIN)
+ //.hintStrategy("sort_merge",
HintPredicates.and(HintPredicates.JOIN, joinWithFixedTableName()))
+ .build();
+ }
+
+ /** Returns a {@link HintPredicate} for join with specified table
references. */
+ private static HintPredicate joinWithFixedTableName()
Review Comment:
Removed this
--
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]
