Skip to content

[Internal] PPAF: Adds Code to Fetch Enablement Flag Through Gateway Database Account Response #5170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 20 commits into from
May 15, 2025

Conversation

NaluTripician
Copy link
Contributor

@NaluTripician NaluTripician commented May 5, 2025

Pull Request Template

Description

This pull request introduces support for partition-level failover in the Cosmos DB SDK. Key changes include updating the connection policy, adding a new property to account settings, and modifying the global endpoint manager to handle the new feature.

Feature: Partition-Level Failover Support

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 #5169

@NaluTripician NaluTripician self-assigned this May 5, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

All good!

@kundadebdatta kundadebdatta self-assigned this May 14, 2025
@kundadebdatta kundadebdatta changed the title Added check for connection policy [Internal] PPAF: Adds Code to Fetch Enablement Flag Through Gateway Database Account Response May 14, 2025
@kundadebdatta kundadebdatta requested a review from Pilchie as a code owner May 14, 2025 21:02
@kundadebdatta kundadebdatta added the auto-merge Enables automation to merge PRs label May 15, 2025
@kirankumarkolli kirankumarkolli merged commit c6b90e4 into master May 15, 2025
26 checks passed
@kirankumarkolli kirankumarkolli deleted the users/nalutripician/ppafGatewayCheck branch May 15, 2025 20:10
Copy link
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 except for few NITs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Enables automation to merge PRs PerPartitionAutomaticFailover
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Per Partition Automatic Failover] - Fetch PPAF Enablement Flag Through Gateway Database Account Response
5 participants