Skip to content

Feature-gating Patch as GA preparation - #5133

Merged
Fabian Meiswinkel (FabianMeiswinkel) merged 6 commits into
mainfrom
users/fabianm/Patch_A
Aug 26, 2026
Merged

Feature-gating Patch as GA preparation#5133
Fabian Meiswinkel (FabianMeiswinkel) merged 6 commits into
mainfrom
users/fabianm/Patch_A

Conversation

@FabianMeiswinkel

Copy link
Copy Markdown
Member

Gating the Patch API behind a new feature-flag preview_patch while iterating on the implementation trade-off (server-side patch vs. cleint-side patch allowing retries due to idempotent exactly-once semantics).

Copilot AI balanced review requested due to automatic review settings August 26, 2026 11:38
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
3 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Introduces an opt-in preview_patch feature for Cosmos point-item PATCH while its retry semantics remain experimental.

Changes:

  • Gates primary SDK and driver PATCH entry points.
  • Conditionally compiles PATCH handlers and tests.
  • Documents preview status, failure semantics, and feature usage.
Show a summary per file
File Description
azure_data_cosmos/tests/emulator_tests/mod.rs Gates PATCH tests.
azure_data_cosmos/tests/emulator_tests/cosmos_hpk.rs Gates HPK PATCH test.
azure_data_cosmos/src/options/mod.rs Gates options export.
azure_data_cosmos/src/options/item.rs Gates and documents PATCH options.
azure_data_cosmos/src/clients/container_client.rs Gates PATCH client method.
azure_data_cosmos/README.md Documents preview usage and risks.
azure_data_cosmos/examples/cosmos_patch.rs Updates example requirements.
azure_data_cosmos/docs/ConfigurationOptions.md Removes PATCH from write-options scope.
azure_data_cosmos/CHANGELOG.md Records SDK breaking change.
azure_data_cosmos/Cargo.toml Defines SDK feature and example requirement.
azure_data_cosmos_driver/tests/framework/test_client.rs Gates test helper.
azure_data_cosmos_driver/tests/emulator_tests/mod.rs Gates driver PATCH tests.
azure_data_cosmos_driver/src/models/cosmos_resource_reference.rs Handles feature-dependent dead code.
azure_data_cosmos_driver/src/models/cosmos_operation.rs Gates driver PATCH APIs.
azure_data_cosmos_driver/src/error/mod.rs Handles feature-dependent dead code.
azure_data_cosmos_driver/src/driver/pipeline/mod.rs Conditionally exposes PATCH pipeline modules.
azure_data_cosmos_driver/src/driver/cosmos_driver.rs Gates PATCH dispatch.
azure_data_cosmos_driver/src/diagnostics/diagnostics_context.rs Handles feature-dependent diagnostics code.
azure_data_cosmos_driver/docs/PATCH_HANDLER_SPEC.md Documents preview failure semantics.
azure_data_cosmos_driver/CHANGELOG.md Records driver breaking change.
azure_data_cosmos_driver/Cargo.toml Defines driver feature.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 21/21 changed files
  • Comments generated: 2
  • Review effort level: Balanced

Comment thread sdk/cosmos/azure_data_cosmos_driver/src/models/cosmos_operation.rs Outdated
Comment thread sdk/cosmos/azure_data_cosmos/Cargo.toml Outdated
Updated changelog to reflect new features and breaking changes in the Azure Data Cosmos SDK.
Updated the changelog to reflect breaking changes and new features related to the Cosmos Driver, including the gating of patch operations behind the 'preview_patch' feature and changes to resource-reference accessors.
Comment thread sdk/cosmos/azure_data_cosmos_driver_native/README.md Outdated

@tvaron3 Tomas Varon (tvaron3) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@FabianMeiswinkel
Fabian Meiswinkel (FabianMeiswinkel) merged commit 9f3a25d into main Aug 26, 2026
13 checks passed
@FabianMeiswinkel
Fabian Meiswinkel (FabianMeiswinkel) deleted the users/fabianm/Patch_A branch August 26, 2026 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cosmos The azure_cosmos crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants