Skip to content

[Internal] Thin Client Integration: Adds support for Per Partition Automatic Failover and Per Partition Circuit Breaker.#5258

Merged
kirankumarkolli merged 17 commits intomasterfrom
users/aavasthy/PPAF_thinclient
Jul 18, 2025
Merged

[Internal] Thin Client Integration: Adds support for Per Partition Automatic Failover and Per Partition Circuit Breaker.#5258
kirankumarkolli merged 17 commits intomasterfrom
users/aavasthy/PPAF_thinclient

Conversation

@aavasthy
Copy link
Copy Markdown
Contributor

@aavasthy aavasthy commented Jun 24, 2025

Pull Request Template

Description

This PR introduces the partition level failover (Automatic Failover for Writes, aka PPAF + Circuit Breaker, aka PPCB) for Thinclient mode. The changes are mainly done in the ThinclientStoreModel which leverages the GlobalPartitionEndpointManager to apply the partition level override for the next account or preferred regions.

A thorough DR drill was conducted for PPAF on the thin proxy accounts, for both Strong and Session consistency levels. Below are the results captured after the DR drill:

  • Account Name: dkunda-tc-strong-account-0701
  • Consistency Level: Strong.
  • Environment: Test14
  • Drill Start Time: 2025-07-05T18:49:17.414Z
  • Drill End Time: 2025-07-05T23:00:17.414Z
  • Accounts Hosted In: Compute Gateway/ Thin Client Federation

Write Failover Analysis:

image

[PPAF: The above image shows the write workload is successfully failing over from West US to East Asia and Failing Back on the reverse path.]

Read Failover Analysis

image

[PPCB: The above image shows the read workload is successfully failing over from West US to East Asia and Failing Back on the reverse path.]

Query Failover Analysis

image

[PPCB: The above image shows the query workload is successfully failing over from West US to East Asia and Failing Back on the reverse path.]

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Closing issues

To automatically close an issue: closes #5247

Copy link
Copy Markdown
Member

@FabianMeiswinkel FabianMeiswinkel left a comment

Choose a reason for hiding this comment

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

LGTM - Thanks

@kundadebdatta
Copy link
Copy Markdown
Member

LGTM. We will need to validate the PPAF with Thin Proxy in Test14/ 61.

@kundadebdatta kundadebdatta marked this pull request as ready for review July 16, 2025 05:26
@kundadebdatta kundadebdatta added the auto-merge Enables automation to merge PRs label Jul 17, 2025
Comment thread Microsoft.Azure.Cosmos/src/ThinClientStoreModel.cs
@kundadebdatta kundadebdatta added auto-merge Enables automation to merge PRs and removed auto-merge Enables automation to merge PRs labels Jul 17, 2025
@kirankumarkolli kirankumarkolli merged commit 6eb976b into master Jul 18, 2025
28 checks passed
@kirankumarkolli kirankumarkolli deleted the users/aavasthy/PPAF_thinclient branch July 18, 2025 02:05
Copy link
Copy Markdown
Member

@FabianMeiswinkel FabianMeiswinkel left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Thin Client Integration] Add Support For Per Partition Automatic Failover

5 participants