cheddar commented on issue #11836: URL: https://github.com/apache/druid/issues/11836#issuecomment-951362719
Yeah, that change is definitely a breach of contract. The design of the cache is to allow the broker to load results for a segment and then not send the query anymore for that segment. This design can be both good and bad, but that's how it works. The adjustment to the cache key from the other change breaks that and should probably be reverted (or put behind a non-default configuration) -- 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]
