It would be nice to have some symmetry between this method and `createUsedSegmentsSet`. Maybe they could both accept a DataSourcesSnapshot and both return a Set. Unifying them under a LoadSegmentStrategy interface as suggested by another comment might be a good way to do that.    What do you think?