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.

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

Reduce memory usage #43

Closed
sebmck opened this issue Feb 8, 2016 · 1 comment
Closed

Reduce memory usage #43

sebmck opened this issue Feb 8, 2016 · 1 comment

Comments

@sebmck
Copy link
Contributor

sebmck commented Feb 8, 2016

When installing react-native we hit peak memory usage of 259.90MB. This should be minimised as much as possible. We could have a disk cache but then we're just reimplementing virtual memory which is silly.
@sebmck
Copy link
Contributor Author

sebmck commented Feb 8, 2016

We hold all request responses in memory until after the package resolution phase which is pretty wasteful. We could probably either:

  1. Not do that.
  2. Only cache the minimum JSON response.

This is wrong, at most our response cache is like 20MB.

@sebmck sebmck closed this as completed Jul 13, 2016
@renovate renovate bot mentioned this issue Feb 2, 2018
Merged
arcanis added a commit that referenced this issue Sep 13, 2018
@arcanis
Merge pull request #43 from arcanis/fix-npm-run
66b2252 
Prevents pnp from bootstrapping when running non-pnp-installed scripts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sebmck