fix: add comment to clarify ignoreUPS variable purpose in DecisionService

FarhanAnjum-opti · FarhanAnjum-opti · commit 2c6a41550754 · 2026-01-12T20:31:28.000+06:00
diff --git a/core-api/src/main/java/com/optimizely/ab/bucketing/DecisionService.java b/core-api/src/main/java/com/optimizely/ab/bucketing/DecisionService.java
@@ -164,7 +164,7 @@ public DecisionResponse<Variation> getVariation(@Nonnull Experiment experiment,
                 return new DecisionResponse(variation, reasons);
             }
         }
-        boolean ignoreUPS = false;
+        boolean ignoreUPS = false; // whether to ignore user profile service for cmab experiments
 
         DecisionResponse<Boolean> decisionMeetAudience = ExperimentUtils.doesUserMeetAudienceConditions(projectConfig, experiment, user, EXPERIMENT, experiment.getKey());
         reasons.merge(decisionMeetAudience.getReasons());

Original file line number	Diff line number	Diff line change
`@@ -164,7 +164,7 @@ public DecisionResponse<Variation> getVariation(@Nonnull Experiment experiment,`
`164`	`164`	`return new DecisionResponse(variation, reasons);`
`165`	`165`	`}`
`166`	`166`	`}`
`167`		`- boolean ignoreUPS = false;`
	`167`	`+ boolean ignoreUPS = false; // whether to ignore user profile service for cmab experiments`
`168`	`168`
`169`	`169`	`DecisionResponse<Boolean> decisionMeetAudience = ExperimentUtils.doesUserMeetAudienceConditions(projectConfig, experiment, user, EXPERIMENT, experiment.getKey());`
`170`	`170`	`reasons.merge(decisionMeetAudience.getReasons());`