-
Notifications
You must be signed in to change notification settings - Fork 733
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
high cpu usage during synchronous exec #167
Comments
@MetaMemoryT: Yeah, that's a side effect of our trying to make @nfischer: Thoughts? |
I think some steps have been taken to reduce cpu usage for newer versions of node. @MetaMemoryT does the issue persist with node v5? |
Is using sleep() from https://www.npmjs.com/package/sleep an option? I appears to use a system call to sleep/stop the process for a given amount of time and it shouldn't burn CPU. |
Isn't this a dead issue thanks to execSync being used in versions of Node that support it (0.12 and better)? Or am I missing something? |
@boutell thanks for pinging this issue. Yes, since we already dropped v0.10 support, I believe this can be safely closed. |
@nfischer Is this note in the README still relevant? Also, is this issue present only on Windows or on Unix platforms as well?
|
@SuhairZain nope 😄 And #825 removes that line. |
Still getting 45% cpu usage when running synchronous exec on windows 8.1
The text was updated successfully, but these errors were encountered: