Skip to content
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.

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

Fix high memory usage (#173) #174

Closed
wants to merge 1 commit into from
Closed

Fix high memory usage (#173) #174

wants to merge 1 commit into from

Conversation

bew
Copy link
Contributor

@bew bew commented Oct 23, 2018

Fixes #173

The toJson() call can use a lot of memory by generating a lot of stats information about modules.
But we don't use this information for this plugin, so we can only enable the generation of assets related data used for this plugin.

Read #173 for more information

@ndelage ndelage mentioned this pull request Dec 3, 2018
Closed
4 tasks
@bew
Copy link
Contributor Author

bew commented Feb 11, 2019

Based on webdeveric/webpack-assets-manifest#47 I simplified the implementation to be future-proof (effectively disabling everything by default instead of manually listing every field we don't want one by one).

@Kenneth-KT
Copy link

Not sure if the author still want to maintain this library.

If not, there is another plugin webpack-assets-manifest that do exactly the same thing and they have this problem fixed in this commit already, but they haven't released it yet.

Run yarn add --dev https://github.com/webdeveric/webpack-assets-manifest.git\#051243e to add it to your repo.

@mastilver
Copy link
Contributor

Hi @bew

Could you fix the tests please?

@mastilver mastilver closed this in b8db85e Sep 25, 2019
Sign up for free to 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.

Very high memory usage (~10GB for ~3000 files)
3 participants