Skip to content

Binary Encoding: Adds support to DateTimeOffset type in CosmosDBNewtonsoftWriter - #5474

Merged
Kiran Kumar Kolli (kirankumarkolli) merged 8 commits into
Azure:masterfrom
Meghana-Palaparthi:5473_add_datetimeoffset_to_cosmosDBToNewtonsoftWriter
Nov 15, 2025
Merged

Binary Encoding: Adds support to DateTimeOffset type in CosmosDBNewtonsoftWriter#5474
Kiran Kumar Kolli (kirankumarkolli) merged 8 commits into
Azure:masterfrom
Meghana-Palaparthi:5473_add_datetimeoffset_to_cosmosDBToNewtonsoftWriter

Conversation

@Meghana-Palaparthi

Copy link
Copy Markdown
Contributor

Description

During some of the internal customer (IC3) validation, it was identified that when the Cosmos .Net SDK is initialized with binary encoding, and a DateTimeOffset field is passed in the payload, the serialization layer is unable to parse the field and hence fails the operation.

This PR fixes the serialization gap by extending support to DateTimeOffset type in CosmosDBNewtonsoftWriter

Type of change

Please delete options that are not relevant.

  • ️✅ Bug fix (non-breaking change which fixes an issue)
  • [] New feature (non-breaking change which adds functionality)
  • [] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [] This change requires a documentation update

Closing issues

To automatically close an issue: closes #5473

@kundadebdatta

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@kundadebdatta

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

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.

@kundadebdatta Debdatta Kunda (kundadebdatta) added auto-merge Enables automation to merge PRs BinaryEncoding binary encoding in .NET sdk labels Nov 10, 2025
@kirankumarkolli

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@kundadebdatta

Copy link
Copy Markdown
Member

/azp run

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 BinaryEncoding binary encoding in .NET sdk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Binary Encoding - Analyze and Fix DateTimeOffset Parsing Issue with .NET SDK Item Operations

3 participants