kfaraz commented on code in PR #18195:
URL: https://github.com/apache/druid/pull/18195#discussion_r2194706647


##########
server/src/main/java/org/apache/druid/client/coordinator/CoordinatorClient.java:
##########
@@ -119,4 +123,36 @@ public interface CoordinatorClient
    * @param tier The name of the tier for which the lookup configuration is to 
be fetched.
    */
   Map<String, LookupExtractorFactoryContainer> fetchLookupsForTierSync(String 
tier);
+
+  /**
+   * Returns an iterator over the metadata segments in the cluster.
+   * <p>
+   * API: {@code GET 
/druid/coordinator/v1/metadata/segments?includeOvershadowedStatus&includeRealtimeSegments}
+   *
+   * @param watchedDataSources Optional datasources to filter the segments by. 
If null or empty, all segments are returned.
+   */
+  ListenableFuture<CloseableIterator<SegmentStatusInCluster>> 
getMetadataSegments(

Review Comment:
   If you look at the implementation of this API on the server-side, the method 
is called `MetadataResource.getAllUsedSegments` since it returns info of used 
segments only.



-- 
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]

Reply via email to