Skip to content
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

run webserver on all available cores, resolves #6 #158

Merged
merged 3 commits into from
Aug 3, 2015
Merged

Conversation

missinglink
Copy link
Member

@heffergm I can see the webserver bottlenecking at 100% CPU on a single core under load, this PR spreads the load across all available cores, which should result in better parallel processing of HTTP requests.

resolves long standing issue: #6 using a module ebay wrote to get parallelism when using the express webserver module.

$ siege -c500 -r10 "http://localhost:3100/search?input=foo"

# single core
# Response time:                0.54 secs
# Response time:                0.52 secs

# multi core
# Response time:                0.16 secs
# Response time:                0.13 secs

@dianashk
Copy link
Contributor

Good call! +100000

@missinglink
Copy link
Member Author

note this may not be compatible with node 0.12, linking pelias/pelias#35

@heffergm
Copy link
Contributor

We run 0.12.2.

@riordan
Copy link
Contributor

riordan commented Jul 14, 2015

Bravo
On Tue, Jul 14, 2015 at 9:39 AM Grant Heffernan [email protected]
wrote:

We run 0.12.2.


Reply to this email directly or view it on GitHub
#158 (comment).

@hkrishna
Copy link
Contributor

👍

@orangejulius
Copy link
Member

nice

@missinglink
Copy link
Member Author

upstream PR opened with ebay: cubejs/cluster2#86

@hkrishna
Copy link
Contributor

👍

@missinglink
Copy link
Member Author

💥 now passing against 0.10 and 0.12 @see: https://travis-ci.org/pelias/api/builds/73865581

missinglink added a commit that referenced this pull request Aug 3, 2015
run webserver on all available cores, resolves #6
@missinglink missinglink merged commit 342e0f1 into master Aug 3, 2015
@orangejulius orangejulius deleted the multicore branch March 26, 2016 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants