Skip to content

[Direct package version update to 3.42.0]: Adds direct package version 3.42.0 and fixes EnableNRegionSynchronousCommit flag#5610

Merged
kirankumarkolli merged 5 commits into
masterfrom
users/aavasthy/nregion
Feb 13, 2026
Merged

[Direct package version update to 3.42.0]: Adds direct package version 3.42.0 and fixes EnableNRegionSynchronousCommit flag#5610
kirankumarkolli merged 5 commits into
masterfrom
users/aavasthy/nregion

Conversation

@aavasthy
Copy link
Copy Markdown
Contributor

@aavasthy aavasthy commented Feb 10, 2026

Pull Request Template

Description

Direct package changes in version 3.42.0:

This PR also fixes the breaking changes for StoreCLient and StoreClientFactory creation as the AccountConfigurationProperties flag got removed.
Refactors N-Region Synchronous Commit feature to use IServiceConfigurationReaderVNext interface instead of AccountConfigurationProperties, enabling Routing Gateway and Compute Gateway to properly consume this account-level feature.

  • Moved EnableNRegionSynchronousCommit from AccountConfigurationProperties to IServiceConfigurationReader.
  • Refactored V3 Code to use the boolean flag from IServiceConfigurationReader.
  • Removed dependency from AccountConfigurationProperties

Test results for end to end test with msdata/direct branch against n-region flag enabled account.
image
image

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

ananth7592
ananth7592 previously approved these changes Feb 11, 2026
Copy link
Copy Markdown
Member

@ananth7592 ananth7592 left a comment

Choose a reason for hiding this comment

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

Change LGTM. If you can attach a sample diagnostic after testing with a real nregion account on test14, it would be a good e2e test.

Comment thread Directory.Build.props
@kirankumarkolli
Copy link
Copy Markdown
Member

  • Please include the Prs direct is brinigng
  • Update title to reflect that tis Direct refresh

@aavasthy aavasthy changed the title [N-Region Synchronous Commit]: Adds EnableNRegionSynchronousCommit to IServiceConfigurationReader [Direct package version update to 3.42.0]: Updates direct package version and adds EnableNRegionSynchronousCommit to IServiceConfigurationReader Feb 12, 2026
@aavasthy aavasthy changed the title [Direct package version update to 3.42.0]: Updates direct package version and adds EnableNRegionSynchronousCommit to IServiceConfigurationReader [Direct package version update to 3.42.0]: Updates direct package version and adds EnableNRegionSynchronousCommit Feb 12, 2026
Copy link
Copy Markdown

@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!

@aavasthy aavasthy changed the title [Direct package version update to 3.42.0]: Updates direct package version and adds EnableNRegionSynchronousCommit [Direct package version update to 3.42.0]: Adds direct package version 3.42.0 and fixes EnableNRegionSynchronousCommit flag Feb 12, 2026
@aavasthy aavasthy added the auto-merge Enables automation to merge PRs label Feb 12, 2026
@aavasthy
Copy link
Copy Markdown
Contributor Author

  • Please include the Prs direct is brinigng
  • Update title to reflect that tis Direct refresh

Added the PR information and updated description.

@kirankumarkolli kirankumarkolli merged commit f3b5ebf into master Feb 13, 2026
33 of 36 checks passed
This was referenced Apr 27, 2026
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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[N-Region Synchronous Commit] ConsistencyWriter - Move EnableNRegionSynchronousCommit from AccountConfigurationProperties to IServiceConfigurationReader

5 participants