-
Notifications
You must be signed in to change notification settings - Fork 63
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
Production Use #1
Comments
Fix typo, pointed out by @maxime-crunding in issue #1.
Thanks! Fixed. |
I want to use this code in production. |
Yes, there's a caveat, see joyent/node issue #7784. I would wait a bit before using it in production, unless the caveat is insignificant for your scenario. In some cases the master process seems to not get the If this won't be fixed in the next couple of days I will rewrite the example in this repository using a different approach. Here's what I have in mind:
|
Thank you the research.
I see a small typo bug.
It should be
var worker = workers[worker_index(connection.remoteAddress, num_processes)];
The scripts works well for me.
The text was updated successfully, but these errors were encountered: