cecemei commented on code in PR #19648:
URL: https://github.com/apache/druid/pull/19648#discussion_r3508742768
##########
server/src/main/java/org/apache/druid/client/CachingClusteredClient.java:
##########
@@ -345,6 +345,19 @@ ClusterQueryResult<T> run(
}
final Set<SegmentServerSelector> segmentServers =
computeSegmentsToQuery(timeline, specificSegments);
+ log.info("Query[%s] found [%d] segments from timeline lookup.",
query.getId(), segmentServers.size());
+ if (query.context().isDebug()) {
Review Comment:
yes it's only logging in debug setting to avoid verbose, i dont think a
query plan has this info? or do you mean defer the logging to sometime later? i
think here's better because it's before the caching/pruning stage so we get a
whole picture of what segments are included from the timeline
--
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]
