-
Notifications
You must be signed in to change notification settings - Fork 7.3k
node 0.10.30 interaction with pm2-interface hangs. OK with node 0.10.29 #8154
Comments
Sorry. I'd put this on the mailing list. Better chance of getting a response there. |
@rragan There was an issue introduced by 0.10.30 that makes timers with non-integer timeouts hang the process. While it doesn't seem that you use timers with non-integer timeouts, the symptoms are pretty similar. See issues #8065 and #8068. A fix for this problem (see #8073) was merged in the v0.10 branch. Could you please try to reproduce the problem with this branch and let us know how it goes? |
I built master branch and the problem is fixed. Seems like the timer change helped whatever was going on. Thanks, Rich From: Julien Gilli [mailto:[email protected]] @rraganhttps://github.com/rragan There was an issue introduced by 0.10.30 that makes timers with non-integer timeouts hang the process. While it doesn't seem that you use timers with non-integer timeouts, the symptoms are pretty similar. See issues #8065#8065 and #8068#8068. A fix for this problem (see #8073#8073) was merged in the v0.10 branch. Could you please try to reproduce the problem with this branch and let us know how it goes? — |
@rragan The master branch never had this issue. The issue was introduced on the v0.10 branch, from which the 0.10.x releases are made. To test if the fix on the v0.10 branch fixes your specific issue, you would have to checkout the v0.10 branch, not master. If you have some time to test that it would be great. Thank you! |
Built again with branch v0.10 and the problem no longer shows up. |
@rragan Thank you for looking into that! |
This is a reduced case from a larger piece of code that hangs with 100% cpu usage by node when pm2 is not installed. The setInterval runs once (usually) and then things hang up at 100% cpu. If pm2 is installed, the code does not hang or if node 0.10.29 and earlier are used it does not hang.
I'm not deliberately trying to use pm2-interface when pm2 is not installed but my code is auto-detecting to see if pm2 is being used or the cluster2 module. The ipm2 line is at the top of the module and is unconditionally executed so I can avoid the problem by moving the line so the execution only happens if pm2 is around. Nevertheless, I wanted to report this because I don't know what changed in node 0.10.30 and whatever did it may pop up in other circumstances since it worked fine in 0.10.29 and earlier. I also reported it to the pm2-interface repo issues.
The text was updated successfully, but these errors were encountered: