capistrant commented on pull request #11429: URL: https://github.com/apache/druid/pull/11429#issuecomment-933553102
> seems reasonable to me. > > What sort of integration test do you think would be needed to validate this change? Ya, I have continued to think about an integration test here but I am not sure there is anything straightforward. In a live cluster, the only way to test that the context override is honored would be by analyzing query metric output or query log output (at least that I can think of). So I plan to instead look at DruidSchema for opportunities to test that the config is honored -- 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]
