abhishekrb19 commented on PR #18329: URL: https://github.com/apache/druid/pull/18329#issuecomment-3228481655
> 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 Hmm yeah, I see quidem tests also use the sql test framework's setup in this case. Actually I think `CachingClusteredClientTest` would be a good candidate. You could add `ServerType.REALTIME` to the `servers` in the test setup and then run some queries with the desired contexts set to true/false. After the query runs, you can verify that `responseContext.remainingResponsesFromQueryServers` has the correct values populated. -- 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]
