J-HowHuang commented on PR #16455: URL: https://github.com/apache/pinot/pull/16455#issuecomment-3234324019
@yashmayya One thing I'm concerning is the load of this periodic task. I added a rate limit set on each controller, for now only one tenant rebalance job allowed at a time. For manual triggered tenant rebalance users are responsible to not schedule multiple tenant rebalances at the same time, but for this periodic task I think some protection is needed here to avoid unexpected huge load when multiple tenant rebalance jobs need retries. Wdyt? Also some loggers are added for observability. -- 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]
