kroeders opened a new pull request #10427: URL: https://github.com/apache/druid/pull/10427
### Description This PR is an extension to provide a solution for https://github.com/apache/druid/issues/10294 where an issue is discussed that queries with lookups may sometimes fail unnecessarily due to the lookup only being available on some historicals. There is a small related change that would be required in core Druid, which is describe in a separate PR here Servers are selected to query for particular segments in CachingClusteredClient using a ServerSelectorStrategy. This extension proposes a new FilteringServerSelectorStrategy that filters the servers provided by the TierSelectorStrategy and then applies another ServerSelectorStrategy (such as the existing random or connection count strategies). A new component, LookupStatusView is added to poll the nodeStatus endpoint on the coordinator for lookup status. This PR has: - [X] been self-reviewed. - [X] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for [code coverage](https://github.com/apache/druid/blob/master/dev/code-review/code-coverage.md) is met. - [ ] been tested in a test Druid cluster. ##### Key changed/added classes in this PR * `FilteringServerSelectorStrategy` * `LookupFilterServerSelectorModule` * `DelegateServerSelectorStrategy` * `LookupAwareFilterStrategy` * `LookupStatusView` * `ServerFilterStrategy` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
