Skip to content

feat(incremental): side effects optimization#8799

Merged
JSerFeng merged 5 commits intoweb-infra-dev:mainfrom
ahabhgk:inc-side-effects
Dec 25, 2024
Merged

feat(incremental): side effects optimization#8799
JSerFeng merged 5 commits intoweb-infra-dev:mainfrom
ahabhgk:inc-side-effects

Conversation

@ahabhgk
Copy link
Copy Markdown
Contributor

@ahabhgk ahabhgk commented Dec 20, 2024

Summary

Incremental side effects optimization

Actually in most cases the improvement of incremental side effects is always not good enough as expected, only improve a little bit, in an internal project (FS): start cost ~63ms and rebuild cost ~44ms for this phase, this is because this only incrementally find optimizable connections, but we didn't/can't implement incremental for do optimize connections and find new optimizable connections based on the optimized connections.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added the release: feature release: feature related release(mr only) label Dec 20, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Dec 20, 2024

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 11003c6
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/676a61eb40387000082ba8d5

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Dec 24, 2024

CodSpeed Performance Report

Merging #8799 will not alter performance

Comparing ahabhgk:inc-side-effects (11003c6) with main (d87bd86)

Summary

✅ 3 untouched benchmarks

@JSerFeng JSerFeng merged commit 93860a8 into web-infra-dev:main Dec 25, 2024
@ahabhgk ahabhgk deleted the inc-side-effects branch December 25, 2024 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: feature release: feature related release(mr only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants