Skip to content

parallelize Descriptions setup, fix response leak, use typed stream keys (#3044) - #3047

Merged
Vincent Biret (baywet) merged 1 commit into
support/v2from
ci/performance-optimization-to-v2
Aug 20, 2026
Merged

parallelize Descriptions setup, fix response leak, use typed stream keys (#3044)#3047
Vincent Biret (baywet) merged 1 commit into
support/v2from
ci/performance-optimization-to-v2

Conversation

@baywet

Copy link
Copy Markdown
Member

port of #3044 to v2

…eys (#3044)
- Dispose HttpResponseMessage in LoadFromUrlAsync (was leaking on every download)
- Run the assembly reads and GHES downloads concurrently in GlobalSetup via
Task.WhenAll instead of sequential awaits; aggregate into _streams afterward
to avoid concurrent Dictionary writes
- Replace string-keyed _streams (mixed file names / URLs) with a private
DescriptionSource enum for compile-time-checked, pre-sized lookups
- Drop the redundant async/await wrapper on each [Benchmark] method so the
MemoryDiagnoser numbers reflect ParseDocumentAsync, not an extra state machine
- Derive the four pinned GHES description URLs from one commit SHA constant
instead of duplicating it across four literals
- Add an HttpClient timeout; modernize Assembly access; align field naming
@github-code-quality

Copy link
Copy Markdown

Code Coverage Overview

Languages: C#

C# / code-coverage/dotnet

The overall line coverage in commit 7ea6c66 in the ci/performance-optim... branch is 41%. The line coverage in commit 9c34bf1 in the support/v2 branch is 89%.

Show a line coverage summary of the most impacted files.
Filesupport/v29c34bf1ci/performance-optim...7ea6c66+/-
/home/runner/wo...MapParameter.cs100%0%-100%
/home/runner/wo...Deserializer.cs81%0%-81%
/home/runner/wo...pyReferences.cs63%31%-32%
/home/runner/wo...piJsonReader.cs100%72%-28%
/home/runner/wo...Deserializer.cs90%62%-28%
/home/runner/wo...s/OpenApiTag.cs100%76%-24%
/home/runner/wo...Deserializer.cs100%81%-19%
/home/runner/wo...Deserializer.cs96%81%-15%
/home/runner/wo...Deserializer.cs97%89%-8%
/home/runner/wo...piComponents.cs94%87%-7%

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate FailedQuality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@baywet
Vincent Biret (baywet) merged commit 77e1ece into support/v2Aug 20, 2026
16 of 17 checks passed
@baywet
Vincent Biret (baywet) deleted the ci/performance-optimization-to-v2 branch August 20, 2026 16:36
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@baywet@adrian05-ms@Imadnajam