J-HowHuang opened a new pull request, #16964: URL: https://github.com/apache/pinot/pull/16964
Currently `PinotHelixResourceManager#getServerInstancesForTable` find instances based on the tenant and tag overrides in table config. This cause problems when: 1. Table config has changed without a rebalance - instances returned by this method are the target servers that don't have any segment yet, also the instances that still host the segments won't be reported. 2. Doesn't reflect tags set in instance assignment config, as well as tier configs - when instance assignment config or tier config is set for the table, the actual instances that are/will be assigned to the table will not be reported. This PR change the method to scan for the current ideal state of the table, and collect all instances that host a segment of the table. -- 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]
