ankitsultana commented on code in PR #14981:
URL: https://github.com/apache/pinot/pull/14981#discussion_r1943839953
##########
pinot-query-planner/src/main/java/org/apache/pinot/calcite/rel/hint/PinotHintOptions.java:
##########
@@ -62,6 +62,7 @@ public static class AggregateOptions {
public static final String GROUP_TRIM_SIZE = "group_trim_size";
public static final String MAX_INITIAL_RESULT_HOLDER_CAPACITY =
"max_initial_result_holder_capacity";
+ public static final String MSE_MAX_INITIAL_RESULT_HOLDER_CAPACITY =
"mse_max_initial_result_holder_capacity";
Review Comment:
@shauryachats : this hint is not required, since the current hint only
controls this value for MSE anyways. (i.e. both of them control the same
thing). We can add a separate hint for this later if required.
--
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
