J-HowHuang opened a new pull request, #19295: URL: https://github.com/apache/pinot/pull/19295
## Description When the rebalance pre-checker was introduced, this `needReload` check was questioned and inconclusive: https://github.com/apache/pinot/pull/15029#discussion_r1953298187 Basically, segments need reload or not on current server doesn't imply whether we need preprocess on the new server that we rebalance to, it depends on if the segment in deep store is up to date with the table config or not. It's irrelevant to rebalance operation's risk. Removing this check as it's misleading. -- 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]
