Skip to content

- Adding nregion feature to changelog#47987

Merged
mbhaskar merged 1 commit intoAzure:mainfrom
mbhaskar:nregion-minor-changes
Feb 17, 2026
Merged

- Adding nregion feature to changelog#47987
mbhaskar merged 1 commit intoAzure:mainfrom
mbhaskar:nregion-minor-changes

Conversation

@mbhaskar
Copy link
Member

Description

This PR does the following:

  • Adding nregion feature to changelog
  • Always parsing nregion header in storeresult

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

- Always parsing nregion header in storeresult
Copilot AI review requested due to automatic review settings February 12, 2026 00:53
@mbhaskar mbhaskar requested review from a team and kirankumarkolli as code owners February 12, 2026 00:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Cosmos Java SDK to better support the N-Region synchronous commit feature by ensuring the N-Region global committed GLSN header is parsed into StoreResult, and documents the feature addition in the Cosmos changelog.

Changes:

  • Always parse x-ms-cosmos-global-nregion-committed-glsn into StoreResult (including for exception responses).
  • Add a changelog entry advertising N-Region synchronous commit support for the upcoming beta release.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
sdk/cosmos/azure-cosmos/src/main/java/com/azure/cosmos/implementation/directconnectivity/StoreReader.java Parse and propagate the N-Region committed GLSN header into StoreResult for the exception path.
sdk/cosmos/azure-cosmos/CHANGELOG.md Document N-Region synchronous commit support under the next unreleased beta version.

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

Copy link
Member

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

@mbhaskar mbhaskar enabled auto-merge (squash) February 13, 2026 03:24
@xinlian12
Copy link
Member

/check-enforcer override

@mbhaskar mbhaskar merged commit 606d04a into Azure:main Feb 17, 2026
44 of 45 checks passed
xinlian12 pushed a commit to xinlian12/azure-sdk-for-java that referenced this pull request Feb 18, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

[AutoPR azure-resourcemanager-managedops]-generated-from-SDK Generation - Java-5788711 (Azure#47788)

* Configurations:  'specification/managedoperations/ManagedOps.Management/tspconfig.yaml', API Version: 2025-07-28-preview, SDK Release Type: beta, and CommitSHA: '83408dfe4894a9b5a5d3989023647bce792efc5f' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5788711 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.

* Configurations:  'specification/managedoperations/ManagedOps.Management/tspconfig.yaml', API Version: 2025-07-28-preview, SDK Release Type: beta, and CommitSHA: 'ed16e10caee8ef2ab09f321272877d0efbec0d1e' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5863942 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.

* Configurations:  'specification/managedoperations/ManagedOps.Management/tspconfig.yaml', API Version: 2025-07-28-preview, SDK Release Type: beta, and CommitSHA: '96e40a96feba1bb2bcabd7d1612feb71f94c49a8' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5877195 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.

* Configurations:  'specification/managedoperations/ManagedOps.Management/tspconfig.yaml', API Version: 2025-07-28-preview, SDK Release Type: beta, and CommitSHA: '816b3edf3aeceb7929a5af05aea47e4a00bf6884' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=5877312 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.

* Update version_client.txt

---------

Co-authored-by: wcas-ms <wcashman@microsoft.com>
Co-authored-by: Weidong Xu <weidxu@microsoft.com>

[Kafka connector]AddSupportForThroughputBucket (Azure#48009)

* add support for throughput bucket in Kafka connector

---------

Co-authored-by: Annie Liang <anniemac@Annies-MacBook-Pro.local>

mgmt, trustedsigning, update to next preview (Azure#48016)

- Adding nregion feature to changelog (Azure#47987)

- Always parsing nregion header in storeresult

Increment package versions for resources releases (Azure#48020)

Remove extra blank line in BulkWriter.scala

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Apply suggestion from @xinlian12

Add fault injection tests for BulkOperationStatusTracker

Add integration tests verifying status code tracking through retry paths:
- TransactionalBulkExecutorTest: inject TOO_MANY_REQUEST with hitLimit(2),
  verify statusTracker records 429 with count=2
- CosmosBulkAsyncTest: inject TOO_MANY_REQUEST for non-transactional bulk,
  verify ItemBulkOperation statusTracker records 429 status codes

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Add CHANGELOG entries for status code history in stale progress logs

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

add more tests and update changelog

add attempt number in BulkOperationFailedException
jeet1995 pushed a commit to jeet1995/azure-sdk-for-java that referenced this pull request Feb 23, 2026
- Always parsing nregion header in storeresult
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants