-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
.Net: Bump Cosmos package and use built-in serializer #8616
Conversation
dotnet/src/Connectors/Connectors.Memory.AzureCosmosDBNoSQL/AzureCosmosDBNoSQLMemoryStore.cs
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Pilchie Thanks for your contribution! Before merging to main
, I will run our integration tests for this connector to double-check whether this change didn't impact anything and will get back to you shortly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can confirm that with this change integration tests are running successfully, and we can merge it to main
branch. @Pilchie thanks again for your contribution!
Thanks! |
Description
Azure Cosmos DB released a new package that allows using System.Text.Json without manually defining a custom serializer. Update to that version and delete the code I copied from the Cosmos package before this was released.