Rename DefinitionsCacheConsumer to SegmentsProvider

Clearer name for the narrow interface that provides the set of segment
names referenced by flag definitions.

AI-Session-Id: 8708ddbc-41d6-47c9-bcb7-9d209ee25aeb
AI-Tool: claude-code
AI-Model: unknown

Author: gthea

client/src/main/java/io/split/storages/SplitCacheCommons.java

@@ -1,5 +1,5 @@
 package io.split.storages;
 
-public interface SplitCacheCommons extends DefinitionsCacheConsumer {
+public interface SplitCacheCommons extends SegmentsProvider {
     long getChangeNumber();
 }

segment-commons/src/main/java/io/split/engine/segments/SegmentSynchronizationTaskImp.java

@@ -1,7 +1,7 @@
 package io.split.engine.segments;
 
 import io.split.engine.common.FetchOptions;
-import io.split.storages.DefinitionsCacheConsumer;
+import io.split.storages.SegmentsProvider;
 import io.split.storages.RuleBasedSegmentCacheConsumer;
 import io.split.storages.SegmentCacheProducer;
 import org.slf4j.Logger;
@@ -36,14 +36,14 @@ public class SegmentSynchronizationTaskImp implements SegmentSynchronizationTask
     private final SegmentCacheProducer _segmentCacheProducer;
     private final ScheduledExecutorService _scheduledExecutorService;
     private final TelemetryListener _telemetryListener;
-    private final DefinitionsCacheConsumer _definitionsCacheConsumer;
+    private final SegmentsProvider _segmentsProvider;
     private final RuleBasedSegmentCacheConsumer _ruleBasedSegmentCacheConsumer;
 
     private ScheduledFuture<?> _scheduledFuture;
 
     public SegmentSynchronizationTaskImp(SegmentChangeFetcher segmentChangeFetcher, long refreshEveryNSeconds, int numThreads,
                                          SegmentCacheProducer segmentCacheProducer, TelemetryListener telemetryListener,
-                                         DefinitionsCacheConsumer definitionsCacheConsumer, ExecutorFactory executorFactory,
+                                         SegmentsProvider segmentsProvider, ExecutorFactory executorFactory,
                                          ThreadFactory threadFactory,
                                          RuleBasedSegmentCacheConsumer ruleBasedSegmentCacheConsumer) {
         _segmentChangeFetcher = Objects.requireNonNull(segmentChangeFetcher);
@@ -57,7 +57,7 @@ public SegmentSynchronizationTaskImp(SegmentChangeFetcher segmentChangeFetcher,
 
         _segmentCacheProducer = Objects.requireNonNull(segmentCacheProducer);
         _telemetryListener = Objects.requireNonNull(telemetryListener);
-        _definitionsCacheConsumer = Objects.requireNonNull(definitionsCacheConsumer);
+        _segmentsProvider = Objects.requireNonNull(segmentsProvider);
         _ruleBasedSegmentCacheConsumer = Objects.requireNonNull(ruleBasedSegmentCacheConsumer);
     }
 
@@ -201,7 +201,7 @@ private void initialize(String segmentName) {
     }
 
     private Set<String> getSegmentNames() {
-        Set<String> names = new HashSet<>(_definitionsCacheConsumer.getSegments());
+        Set<String> names = new HashSet<>(_segmentsProvider.getSegments());
         names.addAll(_ruleBasedSegmentCacheConsumer.getSegments());
 
         return names;

…t/storages/DefinitionsCacheConsumer.java …/io/split/storages/SegmentsProvider.javasegment-commons/src/main/java/io/split/storages/DefinitionsCacheConsumer.java renamed to segment-commons/src/main/java/io/split/storages/SegmentsProvider.java segment-commons/src/main/java/io/split/storages/DefinitionsCacheConsumer.java renamed to segment-commons/src/main/java/io/split/storages/SegmentsProvider.java

@@ -2,6 +2,6 @@
 
 import java.util.Set;
 
-public interface DefinitionsCacheConsumer {
+public interface SegmentsProvider {
     Set<String> getSegments();
 }