Skip to content

Commit 6baf16f

Browse files
authored
ignore experiment status when bucketing rollout rule experiments (#142)
1 parent b378924 commit 6baf16f

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

core-api/src/main/java/com/optimizely/ab/bucketing/DecisionService.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -210,11 +210,7 @@ public DecisionService(@Nonnull Bucketer bucketer,
210210
for (int i = 0; i < rolloutRulesLength - 1; i++) {
211211
Experiment rolloutRule= rollout.getExperiments().get(i);
212212
Audience audience = projectConfig.getAudienceIdMapping().get(rolloutRule.getAudienceIds().get(0));
213-
if (!rolloutRule.isActive()) {
214-
logger.debug("Did not attempt to bucket user into rollout rule for audience \"" +
215-
audience.getName() + "\" since the rule is not active.");
216-
}
217-
else if (ExperimentUtils.isUserInExperiment(projectConfig, rolloutRule, filteredAttributes)) {
213+
if (ExperimentUtils.isUserInExperiment(projectConfig, rolloutRule, filteredAttributes)) {
218214
logger.debug("Attempting to bucket user \"" + userId +
219215
"\" into rollout rule for audience \"" + audience.getName() +
220216
"\".");

0 commit comments

Comments
 (0)