Skip to content

Added Custom Deserializers to Handle Interspersed List Responses #16702

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
Merged
alzimmermsft merged 9 commits into from
Oct 23, 2020
Merged

Added Custom Deserializers to Handle Interspersed List Responses #16702

alzimmermsft merged 9 commits into from
Oct 23, 2020

Conversation

@alzimmermsft
Copy link
Member

@alzimmermsft alzimmermsft commented Oct 22, 2020 •
edited
Loading

Fixes #15414

This PR adds a custom deserializer to PageList to handle the interspersed return of PageRange and ClearRange elements.

@ghost ghost added the Storage Storage Service (Queues, Blobs, Files) label Oct 22, 2020
@alzimmermsft alzimmermsft changed the title Added custom serializer to handle PageList response Added Custom Serializers to Handle Interspersed List Responses Oct 22, 2020
@alzimmermsft alzimmermsft changed the title Added Custom Serializers to Handle Interspersed List Responses Added Custom Deserializers to Handle Interspersed List Responses Oct 22, 2020
@alzimmermsft alzimmermsft marked this pull request as ready for review October 22, 2020 23:17
@gapra-msft
Copy link
Member

gapra-msft commented Oct 23, 2020

Could you add a Changelog item for blob and file share?

@alzimmermsft
Copy link
Member Author

alzimmermsft commented Oct 23, 2020

Could you add a Changelog item for blob and file share?

Added new CHANGELOG entries for the changes.

@alzimmermsft alzimmermsft merged commit 8054a0c into Azure:master Oct 23, 2020
@alzimmermsft alzimmermsft deleted the AzStorage_InterspersedListDeserialization branch October 23, 2020 22:27
@apole apole mentioned this pull request Nov 3, 2020
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gapra-msft gapra-msft gapra-msft approved these changes

@amishra-dev amishra-dev Awaiting requested review from amishra-dev

@jaschrep-msft jaschrep-msft Awaiting requested review from jaschrep-msft

@JonathanGiles JonathanGiles Awaiting requested review from JonathanGiles

@kasobol-msft kasobol-msft Awaiting requested review from kasobol-msft

@mssfang mssfang Awaiting requested review from mssfang

@rickle-msft rickle-msft Awaiting requested review from rickle-msft

Assignees

No one assigned

Labels

Storage Storage Service (Queues, Blobs, Files)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG] Copying incremental snapshot on managed to disk to a page blob not working

2 participants

@alzimmermsft @gapra-msft