Skip to content

[AI-FSSDK] [FSSDK-12735] Add holdout exclusion logic for Targeted Delivery rules - #637

Merged
jaeopt merged 7 commits into
masterfrom
ai/esrakartalOpt/FSSDK-12735-exclude-targeted-delivery
Jul 31, 2026
Merged

[AI-FSSDK] [FSSDK-12735] Add holdout exclusion logic for Targeted Delivery rules#637
jaeopt merged 7 commits into
masterfrom
ai/esrakartalOpt/FSSDK-12735-exclude-targeted-delivery

Conversation

@esrakartalOpt

@esrakartalOptesrakartalOpt commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Implements holdout exclusion logic for Targeted Delivery rules. When a holdout's exclude_targeted_deliveries flag is set to true, users bucketed into that holdout will still be served Targeted Delivery rules normally, while A/B Test and Multi-Armed Bandit rules continue to respect the holdout.

Changes

  • Added excludeTargetedDeliveries field to the Holdout model with parsing support across all four JSON parsers (GSON, Jackson, org.json, json-simple)
  • Updated DecisionService to skip holdout enforcement for delivery rules when the exclusion flag is enabled
  • Preserved existing decision ordering: forced decisions take priority over local holdouts with the exclude flag

Jira Ticket

FSSDK-12735

@jaeoptjaeopt left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@muzahidul-optimuzahidul-opti left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a clarification comments.

@jaeoptjaeopt closed this Jul 29, 2026
@jaeoptjaeopt reopened this Jul 29, 2026
@jaeopt
jaeopt merged commit c506793 into masterJul 31, 2026
27 of 29 checks passed
@jaeopt
jaeopt deleted the ai/esrakartalOpt/FSSDK-12735-exclude-targeted-delivery branch July 31, 2026 16:21
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@esrakartalOpt@jaeopt@muzahidul-opti