Repository navigation
Large number of C++ objects unclaimed in node v0.12.7 #2813
Description
Activity
Have you compared with node v4?
@mscdex - thank you. I would like to test in v4, but: i) 'npm install heapdump' fails with that version of node, ii) I also see couple of outstanding memory leak issue in v4, iii) as I said earlier, we came from v0.10 to resolve some of the issues out there, so would like to work with v0.12.7, and hence this request.
bash-4.1$ npm install heapdump
heapdump@0.3.7 install /home/node/node_modules/heapdump
node-gyp rebuildmake: Entering directory
/home/node/node_modules/heapdump/build' make: Warning: Fileaddon.target.mk' has modification time 53 s in the future
CXX(target) Release/obj.target/addon/src/heapdump.o
In file included from /home/.node-gyp/4.0.0/include/node/node.h:42,
from ../src/heapdump.cc:15:
/home/.node-gyp/4.0.0/include/node/v8.h:336: error: expected unqualified-id before âusingâ@gireeshpunathil it is weird. I can install heapdump without error on v4. Maybe try to remove the node_modules directory and reinstall ?
@gireeshpunathil You need a C++11-capable compiler, i.e., g++ 4.8 or clang++ 3.4 or newer.
@bnoordhuis thanks - I guessed something similar from the compiler error which warns about the C++ syntax.
I will try to fix this. Meanwhile, if the root cause of the leak in 0.12 can be detected, that would be great. Thanks once again!Closing this. Feel free to reopen if this is going to be backported to 0.12 branch or if upgrading to 4.x doesn't fix things.
We just upgraded to node 0.12.7. Our memory usage jumped from a baseline of about 350MB per node to 512MB per node--right at Heroku's max. It's fairly stable, just higher than normal. There are several issues mentioning a memory leak in later versions. What version is now safe? Should we jump to io.js 1.8.2 because the later memory leak is fixed?
Is the amount of memory used higher? Found this stackoverflow.com post about the issue we are experiencing: http://stackoverflow.com/questions/29914837/node-js-0-12-x-memory-usage Said to use --max-old-space-size=512
We can't upgrade because of this issue effecting numerous packages: brianmcd/contextify#180 Seems @rvagg is referenced there as well.
Oh, sorry! I didn't read full thread. Thought, it was a rival of the old discussion.
@base698 I don't know what you do with contextify but you may be able to drop the dependency when you switch to v4.x, the built-in vm module basically works the same as contextify now.
It's other modules we depend on--jsdom being one.
On Sep 22, 2015 3:19 AM, "Ben Noordhuis" notifications@github.com wrote:@base698 https://github.com/base698 I don't know what you do with
contextify but you may be able to drop the dependency when you switch to
v4.x, the built-in vm module basically works the same as contextify now.—
Reply to this email directly or view it on GitHub
#2813 (comment).What version of jsdom are you using? The v4.x releases only work with io.js v3.x and node.js v4.x.
Tried 3.5.0, 4.0.0 on you recommendation just now, and 6.5.0 was the
previous try.npm install jsdom # fails with contextify C++ error mentioned here:
brianmcd/contextify#180 (comment)On Tue, Sep 22, 2015 at 7:25 AM, Ben Noordhuis notifications@github.com
wrote:What version of jsdom are you using? The v4.x releases only work with
io.js v3.x and node.js v4.x.—
Reply to this email directly or view it on GitHub
#2813 (comment).I did a small load test in our testing environment using node 12.7 and 4.1.0 for those interested. It does appear to have way better memory usage in 4.1.0.
At the start of the graph where the vertical black bar is node 4.1.0 was installed. I did a big load test and it looks to have GC'd and behaved fine. Memory flat lined at a pretty low value. We deployed node 0.12.7 at the second vertical black bar and mem usage immediately went high. Doing a small load test put the mem usage even higher than it got with node 4.1.0 during a large load test.
Node version: v0.12.7
Platform tested: Linux IA32