Skip to content

Conversation

@kundadebdatta
Copy link
Member

Pull Request Template

Description

Cherry-pick of following PR from master:

This change adds an ability to override the default value of EnableOptimisticDirectExecution setting using an environment variable named AZURE_COSMOS_OPTIMISTIC_DIRECT_EXECUTION_ENABLED.
The environment variable should have a value that can be parsed as a Boolean in order for the override to work.

As before the default value is true and also the backend setting using clientDisableOptimisticDirectExecution acts as a master switch and acts as global account-level override to disable/enable ODE behavior (in clients).

The environment variable can be used while upgrading the sdk package to 3.38 onwards, which turns on ODE by default. As mentioned in the release notes (of 3.38.0), due to the use of a new type of continuation token, applications that execute queries (and their continuations) in a stateless manner on machines that can be using different versions of sdk, failures can be observed.

This environment variable can be set to true before upgrading the sdk package to avoid such failures during the sdk deployment process in production environment. Once the upgrade is complete the environment variable can be removed or set to false.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Closing issues

To automatically close an issue: closes #IssueNumber

* Code changes to bump up release versions.

* Updated changelog.

---------

Co-authored-by: REDMOND\adityasa <[email protected]>
@kundadebdatta kundadebdatta marked this pull request as ready for review February 7, 2024 00:06
@kundadebdatta kundadebdatta self-assigned this Feb 7, 2024
@kirankumarkolli kirankumarkolli deleted the branch releases/3.38.1 February 7, 2024 00:39
@kundadebdatta kundadebdatta deleted the hotfix/3.38.1 branch February 7, 2024 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants