mayankshriv commented on a change in pull request #6044:
URL: https://github.com/apache/incubator-pinot/pull/6044#discussion_r503636090
##########
File path:
pinot-common/src/main/java/org/apache/pinot/common/utils/CommonConstants.java
##########
@@ -166,6 +166,11 @@
public static final double DEFAULT_BROKER_MIN_RESOURCE_PERCENT_FOR_START =
100.0;
public static final String CONFIG_OF_ENABLE_QUERY_LIMIT_OVERRIDE =
"pinot.broker.enable.query.limit.override";
+ // Config for number of threads to use for Broker reduce-phase.
+ public static final String CONFIG_OF_MAX_REDUCE_THREADS_PER_QUERY =
"pinot.broker.max.reduce.threads";
Review comment:
I debated about it, and felt that sub-setting becomes interesting (e.g.
what does 'per' mean). But seems like there's other configs that also follow
this, so will change.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
