Skip to content

fix(@ngtools/webpack): reduce overhead of Angular compiler rebuild requests#19986

Merged
clydin merged 2 commits intoangular:masterfrom
clydin:rebuild-perf
Feb 8, 2021
Merged

fix(@ngtools/webpack): reduce overhead of Angular compiler rebuild requests#19986
clydin merged 2 commits intoangular:masterfrom
clydin:rebuild-perf

Conversation

@clydin
Copy link
Member

@clydin clydin commented Feb 5, 2021

This change adds additional checks to reduce the number of Webpack rebuildModule calls when the Angular compiler requests additional files to be rebuilt. Now if an emitted file's output does not change from its previous emit, a Webpack rebuild of the module is not performed. This can greatly reduce the amount of computation needed during a rebuild as any files that required re-analysis by the Angular compiler but whose final output did not change will not trigger potential expensive Webpack module graph analysis and additonal module rebuilds.

@clydin clydin added the target: patch This PR is targeted for the next patch release label Feb 5, 2021
@google-cla google-cla bot added the cla: yes label Feb 5, 2021
@clydin clydin marked this pull request as ready for review February 5, 2021 19:03
@clydin clydin requested a review from alan-agius4 February 5, 2021 20:45
…quests

This change adds additional checks to reduce the number of Webpack `rebuildModule` calls when the Angular compiler requests additional files to be rebuilt. Now if an emitted file's output does not change from its previous emit, a Webpack rebuild of the module is not performed. This can greatly reduce the amount of computation needed during a rebuild as any files that required re-analysis by the Angular compiler but whose final output did not change will not trigger potential expensive Webpack module graph analysis and additonal module rebuilds.
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Feb 8, 2021
@clydin clydin merged commit b203378 into angular:master Feb 8, 2021
@clydin clydin deleted the rebuild-perf branch February 8, 2021 14:27
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Mar 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants