gortiz opened a new pull request, #14733:
URL: https://github.com/apache/pinot/pull/14733
This PR fixes an issue on BrokerSelectorUtils when tables on distjoint
brokers are used. Specifically the new test
`BrokerSelectorUtilsTest.getTablesCommonBrokersListTwoTablesDifferentBrokers`
fails in master because `{"broker1"}` was returned in that case.
It also includes some cleanup in the code to reduce the complexity in terms
of computation but also making it easier to read. A bunch of new test have been
added as well
--
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]
