klsince commented on code in PR #9449:
URL: https://github.com/apache/pinot/pull/9449#discussion_r978153541
##########
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/minion/PinotHelixTaskResourceManager.java:
##########
@@ -463,7 +463,7 @@ public synchronized Map<String, PinotTaskConfig>
getSubtaskConfigs(String taskNa
return taskConfigs;
}
- public synchronized Map<String, String> getSubtaskProgress(String taskName,
@Nullable String subtaskNames,
+ public synchronized Map<String, Object> getSubtaskProgress(String taskName,
@Nullable String subtaskNames,
Review Comment:
like commented in last msg, this method was added in my last PR yesterday
and don't think it's used by any yet, this change makes it more easier to be
extended later.
--
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]
