Skip to content

Conversation

@dibahlfi
Copy link
Contributor

@dibahlfi dibahlfi commented Nov 5, 2024

Pull Request Template

Description

This change is to fix CosmosNullReferenceException that happens when a client passes a CancellationToken along with the request. Please see(#4737) for more information.

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

@dibahlfi dibahlfi added auto-merge Enables automation to merge PRs Hedging Any issue/feature request related to request hedging labels Nov 5, 2024
@dibahlfi dibahlfi self-assigned this Nov 5, 2024
kundadebdatta
kundadebdatta previously approved these changes Nov 5, 2024
Copy link
Member

@kundadebdatta kundadebdatta left a comment

Choose a reason for hiding this comment

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

One minor comment. Otherwise, LGTM.

Copy link
Contributor

@NaluTripician NaluTripician left a comment

Choose a reason for hiding this comment

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

A few nits other than that LGTM. Good work, thanks!

@sourabh1007
Copy link
Contributor

We are changing the behavior here, previously if operation was cancelled due to cancellation token, it used to return a valid HedgingResponse, now we are returning exception. Is this expected?

@NaluTripician NaluTripician changed the title CrossRegionHedging: Fixes NullReference Bug CrossRegionHedging: Fixes NullReference Exception Bug Nov 25, 2024
@dibahlfi
Copy link
Contributor Author

dibahlfi commented Dec 1, 2024 via email

@NaluTripician NaluTripician dismissed stale reviews from kirankumarkolli and themself via 242cb11 January 2, 2025 15:07
kirankumarkolli
kirankumarkolli previously approved these changes Jan 2, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot merged commit b6ec4be into master Jan 3, 2025
24 checks passed
@microsoft-github-policy-service microsoft-github-policy-service bot deleted the users/dibahlfi/HedgingCancellationTokenExcpetion branch January 3, 2025 15:30
kundadebdatta pushed a commit that referenced this pull request Jan 8, 2025
# Pull Request Template

## Description

This change is to fix CosmosNullReferenceException that happens when a
client passes a CancellationToken along with the request. Please
see(#4737) for
more information.

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

---------

Co-authored-by: Arooshi Avasthy <[email protected]>
Co-authored-by: Kiran Kumar Kolli <[email protected]>
Co-authored-by: Nalu Tripician <[email protected]>
microsoft-github-policy-service bot pushed a commit that referenced this pull request Jan 8, 2025
# Pull Request Template

## Description

This PR cherry-picks the following commits:

- #4706
- #4869

## Type of change

Please delete options that are not relevant.

- [x] Bug fix (non-breaking change which fixes an issue)
- [x] New feature (non-breaking change which adds functionality)

## Closing issues

To automatically close an issue: closes #IssueNumber

---------

Co-authored-by: Nalu Tripician <[email protected]>
Co-authored-by: Fabian Meiswinkel <[email protected]>
Co-authored-by: Kevin Pilch <[email protected]>
Co-authored-by: dibahlfi <[email protected]>
Co-authored-by: Arooshi Avasthy <[email protected]>
Co-authored-by: Kiran Kumar Kolli <[email protected]>
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 Hedging Any issue/feature request related to request hedging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]:CosmosNullReferenceException when hedging enabled and client called with cancelled cancellation token

7 participants