-
Notifications
You must be signed in to change notification settings - Fork 30k
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
Can we document --max_old_space_size or is it deprecated? #7937
Comments
I believe it is not documented because we have tried to stay away from documenting V8 options. |
Oh, I see it now. |
That's a lot of documentation that would then fall on our shoulders to keep up to date. The V8 project should manage their own documentation IMO. It also looks like that v8-flags repo hasn't been updated in over 2 years. |
@cjihrig Oh, no, I mean to duplicate just the link. But if the list is outdated. then may be this is not a good idea. Sorry. |
Ah, sorry, I misunderstood what you meant by "duplicate it." I wouldn't have a strong opinion on duplicating the link if the v8-flags repo was being kept up to date. |
+1 on at least mentioning something about max_old_space_size in the docs. If only to link to somewhere else.
I agree that it's not Nodes' job to document V8 options, but still believe this memory option is a special case that deserve mentioning in the docs. Which would be best to use:
or
Edit: |
@fractalf A posteriori, Node.js sets memory limit near 1.5 GB by default (at least for x64 OS). |
@fractalf FWIW the definitions of default values are in v8/src/heap/heap.cc: Line 82 in ec02b81
max_old_space_size is actually used to configure max_old_generation_size when its value is larger than 0). So on 64-bit machines that(the old generation alone) would be 1400 MB.
|
@vsemozhetbyt @joyeecheung Thanks! |
webpack for build-min is called with --max_old_space_size=2048 which seems to make travis work again, but we don't know exactly what this flag is doing, and what was the default value that we were using. nodejs/node#7937 This commit also clean some vestigial code from .travis.yaml that is not needed now that we test inside the docker container.
webpack for build-min is called with --max_old_space_size=2048 which seems to make travis work again, but we don't know exactly what this flag is doing, and what was the default value that we were using. nodejs/node#7937 This commit also clean some vestigial code from .travis.yaml that is not needed now that we test inside the docker container.
webpack for build-min is called with --max_old_space_size=2048 which seems to make travis work again, but we don't know exactly what this flag is doing, and what was the default value that we were using. nodejs/node#7937 This commit also clean some vestigial code from .travis.yaml that is not needed now that we test inside the docker container.
"scripts": { instead of: "scripts": { |
It is now documented under the Command-Line options in the Docs:
|
I have some Node.JS processes taking more memory than I think they require. I suspect this is due to lazy GC.
In my case, I have several Node.JS processes running, across 3 user accounts. (
root
and two others)After searching, it seems that I can limit the amount of 'unused' space that is held in memory by simply using the
--max_old_space_size=X
option. WhereX
is the number of megabytes allowed.However, this does not seem to be documented so I am a little concerned that I should not be using it. Can this be documented, or is it deprecated?
The text was updated successfully, but these errors were encountered: