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.

Sign up for GitHub

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

Jump to bottom

[Internal] Thin Client Integration: Adds Transport Serialization of RNTBD Payload #4637

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

[Internal] Thin Client Integration: Adds Transport Serialization of RNTBD Payload #4637

wants to merge 4 commits into from
+400 −1

Conversation

aavasthy
Copy link
Contributor

Pull Request Template

Description

The ThinClientTransportSerializer class provides methods for serializing and deserializing proxy requests and responses. It serializes HttpRequestMessage objects into RNTBD requests by extracting headers, operation types, resource types, and partition keys, and it deserializes RNTBD responses back into HttpResponseMessage objects, reconstructing headers and body content. This change also includes unit tests for the change.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Closing issues

To automatically close an issue: closes #4582

github-actions[bot]
github-actions bot reviewed Aug 12, 2024
View reviewed changes
Copy link

@github-actions github-actions bot left a comment •
edited
Loading

Choose a reason for hiding this comment

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

All good!

@aavasthy aavasthy changed the title [Thin Client Integration] Add Transport Serialization of RNTBD Payload [Internal] [Thin Client Integration] Adds Transport Serialization of RNTBD Payload Aug 12, 2024
@aavasthy aavasthy changed the title [Internal] [Thin Client Integration] Adds Transport Serialization of RNTBD Payload [Internal]: [Thin Client Integration] Adds Transport Serialization of RNTBD Payload Aug 12, 2024
@aavasthy aavasthy changed the title [Internal]: [Thin Client Integration] Adds Transport Serialization of RNTBD Payload [Internal] Thin Client Integration: Adds Transport Serialization of RNTBD Payload Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] left review comments

@khdang khdang Awaiting requested review from khdang khdang will be requested when the pull request is marked ready for review khdang is a code owner

@sboshra sboshra Awaiting requested review from sboshra sboshra will be requested when the pull request is marked ready for review sboshra is a code owner

@adityasa adityasa Awaiting requested review from adityasa adityasa will be requested when the pull request is marked ready for review adityasa is a code owner

@neildsh neildsh Awaiting requested review from neildsh neildsh will be requested when the pull request is marked ready for review neildsh is a code owner

@kirankumarkolli kirankumarkolli Awaiting requested review from kirankumarkolli kirankumarkolli will be requested when the pull request is marked ready for review kirankumarkolli is a code owner

@FabianMeiswinkel FabianMeiswinkel Awaiting requested review from FabianMeiswinkel FabianMeiswinkel will be requested when the pull request is marked ready for review FabianMeiswinkel is a code owner

@kirillg kirillg Awaiting requested review from kirillg kirillg will be requested when the pull request is marked ready for review kirillg is a code owner

@Pilchie Pilchie Awaiting requested review from Pilchie Pilchie will be requested when the pull request is marked ready for review Pilchie is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@aavasthy aavasthy

Labels
None yet
Projects
Azure Cosmos SDKs
Status: Approved
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Thin Client Integration] Create Classes and Methods for Transport Serialization of RNTBD Payload
1 participant
@aavasthy