J-HowHuang commented on PR #16455: URL: https://github.com/apache/pinot/pull/16455#issuecomment-3237617108
@yashmayya > Also, just to clarify, we shouldn't be retrying the same tenant rebalance job anyway right (across controllers as well as on the same controller)? Right, the mechanics using ZK metadata update prevent this. I wrote that one comment in a wrong way. IMO to schedule them one by one is safer, plus that if multiple tenant rebalance jobs are to retry, the one retry per controller protection can distribute retries job to different controllers (though some will have to wait if the number of jobs > number of controllers), compared to without the one job limit, one controller could take up every tenant rebalance jobs to retry because we have no leadership in tenant level like tables. -- 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]
