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

Performance #18

Closed
bassomatic32 opened this issue Feb 25, 2013 · 3 comments
Closed

Performance #18

bassomatic32 opened this issue Feb 25, 2013 · 3 comments

Comments

@bassomatic32
Copy link

I finally got the opportunity to update my dependency from the 'memjs-concurrent' package to the latest 'memjs' version.

Unfortunately, my QA guys quickly discovered an over 50% drop in performance in one of our tests directly related to caching. Reverting back to the 'memjs-concurrent' brought performance of that test back to nominal levels.

The test involves 30 concurrent requests over 4 nodes on Heroku with a Memcachier backed cache. The nature of the test is such that each request will pull the same item from the cache. This item is probably on the order of 10K in size.

I wish I had more data. I don't expect to have time in the near future to do a more comprehensive and isolated test.

@alevy
Copy link
Member

alevy commented Feb 25, 2013

Thanks, I'll definitely look into this. Will update the ticket as I find more.

@alevy
Copy link
Member

alevy commented Mar 1, 2013

@bassomatic32 I'm having trouble reproducing this. In particular, in my benchmarks the current version of memjs is actually faster. Is there some code you can share to help reproduce?

@bassomatic32
Copy link
Author

Unfortunately, no. It's all part of a more complex system.

Michael Bass
(mobile)

On Feb 28, 2013, at 5:28 PM, Amit Levy notifications@github.com wrote:

@bassomatic32 I'm having trouble reproducing this. In particular, in my benchmarks the current version of memjs is actually faster. Is there some code you can share to help reproduce?


Reply to this email directly or view it on GitHub.

@alevy alevy closed this as completed Sep 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants