aho135 commented on PR #18329: URL: https://github.com/apache/druid/pull/18329#issuecomment-3226303084
> Jacaco code coverage check is failing. Looks like most of the sql tests use `TestClusterQuerySegmentWalker` instead of the production class `CachingClusteredClient`. Is it possible to add a test exercising this code path in `CachingClusteredClient` - maybe a quidem test should help? I was just testing this out, but found that quidem tests are also using TestClusterQuerySegmentWalker. Any other suggestions on how to test this code path? FWIW it does seem like other query context properties are in a similar state -- 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]
