Skip to content

[StaticWebAssets] Backport perf fixes #47913

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
merged 8 commits into from
Apr 21, 2025
Merged

[StaticWebAssets] Backport perf fixes #47913

merged 8 commits into from
Apr 21, 2025

Conversation

javiercn
Copy link
Member

@dotnet-issue-labeler dotnet-issue-labeler bot added Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch untriaged Request triage from a team member labels Mar 26, 2025
@javiercn javiercn changed the base branch from main to release/9.0.3xx March 26, 2025 15:00
@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

Thanks for your PR, @javiercn.
To learn about the PR process and branching schedule of this repo, please take a look at the SDK PR Guide.

This was referenced Mar 26, 2025
Open
Open
javiercn added 8 commits April 21, 2025 18:05
@javiercn
[StaticWebAssets] Process endpoint definitions in parallel (#43736)
c86d8c2
@javiercn
[StaticWebAssets] Improve globbing match performance (#44159)
3287d34 
* Removes the usage of Microsoft.Extensions.FileSystemGlobbing in favor
  of a custom implementation optimized for our scenarios.
* Improves the parallelism in DefineStaticWebAssetEndpoints.
* Spanifies the string manipulation logic.
@javiercn
[StaticWebAssets] Add file details to SWA
27e08ea 
* Ensures that only DefineStaticWebAssets hits disk
* Avoids doing extra work on tasks that need to use the length or the last write time.
* Simplifies other tasks like DefineStaticWebAssetEndpoints and ApplyCompressionNegotiation
@javiercn
[StaticWebAssets] Adds incrementalism to DefineStaticWebAssets (#47701)
dfb5379 
* Makes the task incremental and retrieves the results from a cache to avoid hitting disk to compute integrity and fingerprint on incremental builds
@javiercn
Fix build
b9906d6
@javiercn
Cleanup port
e3a52ea
@javiercn
cleanups
5441186
@javiercn
Fix baselines
f657d38
@javiercn javiercn force-pushed the javiercn/backport-perf-fixes branch from 0b8a077 to f657d38 Compare April 21, 2025 16:05
@javiercn javiercn requested a review from MackinnonBuck April 21, 2025 16:05
@javiercn javiercn marked this pull request as ready for review April 21, 2025 16:05
@javiercn javiercn requested review from a team as code owners April 21, 2025 16:05
@javiercn javiercn merged commit d79ec02 into release/9.0.3xx Apr 21, 2025
30 checks passed
@javiercn javiercn deleted the javiercn/backport-perf-fixes branch April 21, 2025 17:48
@build-analysis build-analysis bot mentioned this pull request Apr 21, 2025
Open
@v-wuzhai v-wuzhai mentioned this pull request Apr 30, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MackinnonBuck MackinnonBuck MackinnonBuck approved these changes

Assignees
No one assigned
Labels
Area-AspNetCore RazorSDK, BlazorWebAssemblySDK, dotnet-watch untriaged Request triage from a team member
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@javiercn @MackinnonBuck