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


##########
server/src/main/java/org/apache/druid/indexing/overlord/IndexerMetadataStorageCoordinator.java:
##########
@@ -473,4 +473,22 @@ SegmentPublishResult commitMetadataOnly(
    * @return List of pending segment records
    */
   List<PendingSegmentRecord> getPendingSegments(String datasource, Interval 
interval);
+
+  /**
+   * Map from a segment ID to the segment ID from which it was upgraded
+   * There should be no entry in the map for an original non-upgraded segment
+   * @param dataSource data source
+   * @param segmentIds ids of segments
+   * @return map from child id to parent id
+   */
+  Map<String, String> retrieveUpgradedFromSegmentIds(String dataSource, 
Set<String> segmentIds);
+
+  /**
+   * Map from a segment ID to a set containing
+   * all segment IDs that were upgraded from it AND are still present in the 
metadata store
+   * @param dataSource data source
+   * @param segmentIds ids of the first segments which had the corresponding 
load spec
+   * @return map from parent id to set of all its children

Review Comment:
   Not needed.



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