swaminathanmanish commented on code in PR #16571:
URL: https://github.com/apache/pinot/pull/16571#discussion_r2272360750
##########
pinot-core/src/main/java/org/apache/pinot/core/common/MinionConstants.java:
##########
@@ -63,18 +63,25 @@ private MinionConstants() {
public static final String TIMEOUT_MS_KEY_SUFFIX = ".timeoutMs";
public static final String NUM_CONCURRENT_TASKS_PER_INSTANCE_KEY_SUFFIX =
".numConcurrentTasksPerInstance";
public static final String MAX_ATTEMPTS_PER_TASK_KEY_SUFFIX =
".maxAttemptsPerTask";
+ // Cluster level config of maximum subtasks for a given task
+ // This is primarily used to prevent performance issues in helix leader
controller when it creates
+ // more subtasks than it can support
+ public static final String MAX_ALLOWED_SUB_TASKS_KEY =
"maxAllowedSubTasksPerTask";
Review Comment:
Given this is cluster config, please name this to indicate its minion
config.
--
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]
