Jackie-Jiang commented on code in PR #13509:
URL: https://github.com/apache/pinot/pull/13509#discussion_r1663333198
##########
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/minion/PinotTaskManager.java:
##########
@@ -702,6 +702,14 @@ public void cleanUpTask() {
}
}
+ @Override
+ protected void nonLeaderCleanup(List<String> tableNamesWithType) {
+ LOGGER.info("Cleaning up all task generators for tables that the
controller is not the leader for.");
Review Comment:
Consider logging the tables? You may pick the first 10 tables if there are
too many
--
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
