[GitHub] [druid] kroeders commented on issue #10294: Failed Query due to missing lookup on some servers

[GitBox <gi...@apache.org>]

kroeders commented on issue #10294:
URL: https://github.com/apache/druid/issues/10294#issuecomment-697986177


   Hi @a2l007 @jihoonson I was wondering if you could have a look at the approach in the linked pull requests when you have a chance? The basic approach is to add an extension that provides a new ServerSelectorStrategy that filters out servers without related lookups. There would be one small change to the core code, which would be to add the Query as an optional parameter through the server selection codepath. The advantage is that a new ServerSelectorStrategy can be registered that applies a filter to the servers coming from TierSelectorStrategy and then delegates to an existing ServerSelectorStrategy, so there is minimal overhead and potentially the same approach could be extended to filter for other reasons. I don't think there is much overhead to this change and I'm going to test it on our staging cluster in the next few days. Thanks!


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org