Update client/src/main/java/io/split/engine/experiments/SplitFetcherImp.java

chillaq · sanzmauro · web-flow · commit efd26059d8f3 · 2025-04-15T09:28:16.000-07:00
Co-authored-by: Mauro Sanz &lt;51236193+sanzmauro@users.noreply.github.com&gt;
diff --git a/client/src/main/java/io/split/engine/experiments/SplitFetcherImp.java b/client/src/main/java/io/split/engine/experiments/SplitFetcherImp.java
@@ -158,10 +158,8 @@ private Set<String> runWithoutExceptionHandling(FetchOptions options) throws Int
         return segments;
     }
 
-    private boolean checkExitConditions(SplitChange change) {
-        return ((change.featureFlags.s != _splitCacheProducer.getChangeNumber() || change.featureFlags.t < _splitCacheProducer.getChangeNumber())
-                || (change.ruleBasedSegments.s != _ruleBasedSegmentCacheProducer.getChangeNumber() ||
-                    change.ruleBasedSegments.t < _ruleBasedSegmentCacheProducer.getChangeNumber()));
+    private <T> boolean checkExitConditions(ChangeDto<T> change, long cn) {
+        return change.s != cn || change.t < cn;
     }
 
     private boolean checkReturnConditions(SplitChange change) {

Original file line number	Diff line number	Diff line change
`@@ -158,10 +158,8 @@ private Set<String> runWithoutExceptionHandling(FetchOptions options) throws Int`
`158`	`158`	`return segments;`
`159`	`159`	`}`
`160`	`160`
`161`		`- private boolean checkExitConditions(SplitChange change) {`
`162`		`- return ((change.featureFlags.s != _splitCacheProducer.getChangeNumber() \|\| change.featureFlags.t < _splitCacheProducer.getChangeNumber())`
`163`		`- \|\| (change.ruleBasedSegments.s != _ruleBasedSegmentCacheProducer.getChangeNumber() \|\|`
`164`		`- change.ruleBasedSegments.t < _ruleBasedSegmentCacheProducer.getChangeNumber()));`
	`161`	`+ private <T> boolean checkExitConditions(ChangeDto<T> change, long cn) {`
	`162`	`+ return change.s != cn \|\| change.t < cn;`
`165`	`163`	`}`
`166`	`164`
`167`	`165`	`private boolean checkReturnConditions(SplitChange change) {`