Skip to content

[Internal] Thin Client Integration: Fixes Exclude regions check for thinclient mode.#5928

Open
aavasthy wants to merge 2 commits into
mainfrom
users/aavasthy/thinclienthedging
Open

[Internal] Thin Client Integration: Fixes Exclude regions check for thinclient mode.#5928
aavasthy wants to merge 2 commits into
mainfrom
users/aavasthy/thinclienthedging

Conversation

@aavasthy
Copy link
Copy Markdown
Contributor

@aavasthy aavasthy commented Jun 5, 2026

Pull Request Template

Description

This change resolves an issue where per-request region exclusions were not honored when the SDK operated in thin client mode. Previously, requests configured with ExcludeRegions would still route to theprimary thin client endpoint regardless of the customer's regional preferences, preventing them from steering traffic away from specific regions during operational events. With this fix, thin client requestrouting now filters available endpoints against the excluded region list in a manner consistent with the standard gateway path, restoring expected behavior for both individual operations and cross-region hedging scenarios. The change is scoped narrowly to the thin client resolution path, preserves all existing routing semantics when no exclusions are specified, and is covered by new unit and integration test.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Closing issues

To automatically close an issue: closes #5789

Changelog

  • I have added a changelog entry under ### Unreleased in changelog.md
    for the user-facing impact of this change.

If the second box is checked, briefly justify here:

Copy link
Copy Markdown
Member

@kushagraThapar kushagraThapar left a comment

Choose a reason for hiding this comment

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

Thanks @aavasthy, overall looks good, it would be good to have some more testing on all regions excluded code path and end to end hedging tests?

Sign up for free to 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.

ExcludeRegions on ItemRequestOptions is ignored when thin client mode is enabled

2 participants