Skip to content

[release/5.0-preview8] Create and add instance reference inside ResolveMetadata* (#39330)#39645

Merged
jozkee merged 1 commit intodotnet:release/5.0-preview8from
jozkee:json_port_39330
Jul 21, 2020
Merged

[release/5.0-preview8] Create and add instance reference inside ResolveMetadata* (#39330)#39645
jozkee merged 1 commit intodotnet:release/5.0-preview8from
jozkee:json_port_39330

Conversation

@jozkee
Copy link
Member

@jozkee jozkee commented Jul 20, 2020

Description

Port of #39330

Customer Impact

No functional changes or bugs fixed but this grants an slight perf. boost for stream-based scenarios.

Risk

Low, changes only affect scenarios that use ReferenceHandler or JsonSerializer.DeserializeAsync APIs.

cc @jeffhandley

* Create and add instance reference on ResolveMetadata

* Address suggestions
@jozkee jozkee requested a review from layomia July 20, 2020 19:20
@jozkee jozkee requested a review from steveharter as a code owner July 20, 2020 19:20
@jozkee jozkee self-assigned this Jul 20, 2020
@jozkee jozkee added this to the 5.0.0 Preview 8 milestone Jul 20, 2020
@jozkee jozkee added the Servicing-consider Issue for next servicing release review label Jul 20, 2020
@jozkee jozkee merged commit 9a0b34d into dotnet:release/5.0-preview8 Jul 21, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 2020
@jozkee jozkee deleted the json_port_39330 branch March 24, 2021 19:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Text.Json Servicing-consider Issue for next servicing release review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants