vvivekiyer commented on code in PR #10315:
URL: https://github.com/apache/pinot/pull/10315#discussion_r1113436552
##########
pinot-query-planner/src/main/java/org/apache/calcite/rel/hint/PinotHintStrategyTable.java:
##########
@@ -16,19 +16,21 @@
* specific language governing permissions and limitations
* under the License.
*/
-package org.apache.pinot.query.planner.hints;
-
-import org.apache.calcite.rel.hint.RelHint;
-
+package org.apache.calcite.rel.hint;
/**
- * Provide certain relational hint to query planner for better optimization.
+ * Default hint strategy set for Pinot query.
*/
-public class PinotRelationalHints {
- public static final RelHint AGG_INTERMEDIATE_STAGE =
RelHint.builder("AGG_INTERMEDIATE_STAGE").build();
- public static final RelHint AGG_LEAF_STAGE =
RelHint.builder("AGG_LEAF_STAGE").build();
+public class PinotHintStrategyTable {
+ public static final String INTERNAL_AGG_INTERMEDIATE_STAGE =
"aggIntermediateStage";
+ public static final String INTERNAL_AGG_FINAL_STAGE = "aggFinalStage";
Review Comment:
nit: can we convert this to a static class for each hint? That way we'll be
able to colocate all properties for each hint like name, HintPredicate, Options
Checker, etc?
I was thinking something similar to the code here -
https://github.com/apache/pinot/pull/10248/files#diff-5d04e3ba2f89fa9f8a646c2a24f79901f0eb107cfc4ccbf574d78213a614b756
--
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...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
