walterddr commented on code in PR #11036:
URL: https://github.com/apache/pinot/pull/11036#discussion_r1253755083
##########
pinot-query-planner/src/main/java/org/apache/calcite/rel/hint/PinotHintOptions.java:
##########
@@ -29,12 +29,29 @@ public class PinotHintOptions {
public static final String AGGREGATE_HINT_OPTIONS = "aggOptions";
public static final String JOIN_HINT_OPTIONS = "joinOptions";
+ /**
+ * Hint to denote that the aggregation node is the final aggregation stage
which extracts the final result.
+ */
+ public static final String INTERNAL_AGG_OPTIONS = "aggOptionsInternal";
+
private PinotHintOptions() {
// do not instantiate.
}
+ public static class InternalAggregateOptions {
+ public static final String AGG_TYPE = "agg_type";
+ public enum AggType {
+ DIRECT,
+ LEAF,
+ INTERMEDIATE,
+ FINAL,
+ REDUCE // NOT SUPPORTED
+ }
Review Comment:
these are the only necessary MODE we needed
--
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]
