I have a few changes to support MSE adaptive routi...
# pinot-devt
Tim Elgersma
07/20/2026, 3:55 PMI have a few changes to support MSE adaptive routing.
• Add latency tracking for MSE adaptive routing
• Make adaptive routing variables runtime adjustable
• [#12507] Implement adaptive pool selection for StrictReplicaGroupInstanceSelector - fixes correctness bugs with SSE adaptive routing + upsert tables
@Jia Guo / @Sidd / @Vivek Iyer Vaidyanathan / @Xiang Fu can you TAL at these? They've been deployed on Stripe's clusters for a few weeks / months and working well.
👀 1
m
Mayank
07/20/2026, 5:22 PM
cc @Gonzalo Ortiz @Yash Mayya
t
Tim Elgersma
07/23/2026, 9:11 PM@Xiang Fu @Gonzalo Ortiz @Yash Mayya are any of you able to take a look at these?
g
Gonzalo Ortiz
07/27/2026, 10:49 AMI didn’t. I’ll take a look this week
t
Tim Elgersma
08/24/2026, 4:25 PMHey @Gonzalo Ortiz, I've addressed your comments on [#12507] Implement adaptive pool selection for StrictReplicaGroupInstanceSelector, do you have any other thoughts?
g
Gonzalo Ortiz
08/28/2026, 3:59 PMI missed the message and it is getting late in EU. I’m going to review that early next week
thanks 1
Gonzalo Ortiz
08/31/2026, 4:57 PMHi Tim! I’ve approved the PR and added some comments. ANyway, we need to resolve some conflicts
Gonzalo Ortiz
08/31/2026, 4:58 PMI’ll merge the PR as soon as the merge is fixed. The comments are probably not that problematic.
t
Tim Elgersma
08/31/2026, 8:50 PMThanks for the review! I rebased and addressed most of the comments (the merge conflicts were simple, and my latest changes are in the last 3 commits).
Tim Elgersma
08/31/2026, 8:52 PMDo you think you can help me get github.com/apache/pinot/pull/18791 reviewed next? In it's current state it's being tested at Stripe's scale, but as the base drifts away from what Stripe is running it'll get harder to re-test.
2 Views