bytelyst/learning_ai_common_plat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(broadcasts): Fix percentage rollout bucketing logic

Browse Source 
- Use userId only as seed instead of userId + percentage
- Prevents users from being re-bucketed when rollout % increases
- Ensures deterministic assignment for phased rollouts
This commit is contained in:
saravanakumardb1 2026-03-02 23:56:42 -08:00
parent e4681fbbb2
commit ecb9f67c99
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
services/platform-service/src/modules/broadcasts/targeting.ts
View File

@ -79,8 +79,9 @@ export function evaluateTarget(
// Percentage rollout (FNV-1a hash for deterministic bucketing) // Percentage rollout (FNV-1a hash for deterministic bucketing)
if (target.percentageRollout !== undefined && target.percentageRollout < 100) { if (target.percentageRollout !== undefined && target.percentageRollout < 100) {
// Hash userId + broadcast seed for consistent assignment // Use userId only as seed for consistent bucketing
const hash = fnv1a32(`${context.userId}:${target.percentageRollout}`); // This ensures when rollout % increases, same users stay included
const hash = fnv1a32(context.userId);
const bucket = hash % 100; const bucket = hash % 100;
if (bucket >= target.percentageRollout) { if (bucket >= target.percentageRollout) {
return false; return false;