Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CosmosDB] Upgraded SDK to the latest version #1628

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

RaviTella
Copy link

@RaviTella RaviTella commented Jul 27, 2022

This includes:

  1. Cosmos DB SDK upgrade to the latest version
  2. A change to allow for request diagnostic logging
  3. Update operation uses the new patch API instead of read and replace
  4. removed retry capability

This Pull requests is a super set of an open pull request #1606 made my teammate simplynaveen20.

1) SDK upgrade to 4.28.0
2) Update operation with Patch
3) removed Retry
4) updated error log
5) Adding status code for errors
6)Adding diagnosticsLatencyThresholdInMS
adding request diagnostics configuration
@RaviTella RaviTella changed the title User/ratella/cosmos update [CosmosDB] Upgraded SDK to the latest version Jul 27, 2022
/**
* Azure Cosmos DB Java SDK 4.6.0 client for YCSB.
* Azure Cosmos DB Java SDK 4.28.0 client for YCSB.

Choose a reason for hiding this comment

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

4.28.0 -> 4.34.0

Copy link
Author

Choose a reason for hiding this comment

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

Good catch. Change pushed to the same pull request!

Copy link

@simplynaveen20 simplynaveen20 left a comment

Choose a reason for hiding this comment

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

LGTM

@RaviTella RaviTella closed this by deleting the head repository Sep 9, 2022
@RaviTella RaviTella reopened this Oct 21, 2022
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.

2 participants