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

GetMulti is broken by NewRelic #226

Closed
ant0ine opened this issue Dec 30, 2014 · 2 comments
Closed

GetMulti is broken by NewRelic #226

ant0ine opened this issue Dec 30, 2014 · 2 comments

Comments

@ant0ine
Copy link

ant0ine commented Dec 30, 2014

After playing with the node debugger for a while I found that NR hijacks the memcached.command function and manages to drop the last arg (the server arg) causing the client to not dispatch the getMulti calls correctly (all calls sent to the same server)

So, this client code is fine, this is just a FYI if you use NewRelic. I'll dig deeper and open an issue on the NR lib.

@3rd-Eden
Copy link
Owner

@ant0ine Yes, it's probably best if you report this at NewRelic as I can't really prevent these libraries from breaking the code.

@ant0ine
Copy link
Author

ant0ine commented Dec 30, 2014

@apparentlymart who works with me at SAY fixed it: newrelic/node-newrelic#190

It would be nice if NR could run your unit tests with their integration.

I'm closing this issue.

@ant0ine ant0ine closed this as completed Dec 30, 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