Error reporting - Node.js-style callbacks and custom logger about memjs HOT 3 CLOSED

Thanks for the comments! I'm going to break them up into separate issues so they can be addressed and closed independently, but also responding here:

1. For the (err) parameter, this makes sense in general. Worth discussing this more in depth in a separate issue, specifically when/which errors to pass in specific cases.
2. I disagree that errors should be bubbled up rather than logged. Definitely for certain errors (like failure of operations that will be retried), I think it's important for the caller to be oblivious to this happening (notice that in the case of retries it's still a user tunable parameter). I do agree that some errors (e.g. out of memory) should bubble up as well.
3. RE: console logging. Again, this makes sense to me. The one caveat is that with memcache you often want to silently ignore failures (as long as they are transient) but log them so you know. I'm happy allowing other logging mechanisms as long as it doesn't complicate the common case. Perhaps just a config var that's a logging function the user can replace? Would just default to console.log...

Anyway, I'll create some specific issue around these comments, please feel free to comment/discuss/send pull requests on those as well. Thanks!

from memjs.

Marking as duplicate since I broke this into two separate issues.

from memjs.

Of course, thanks!
On Jun 19, 2012 10:12 PM, "Amit Levy" <
[email protected]>
wrote:

Marking as duplicate since I broke this into two separate issues.

---

Reply to this email directly or view it on GitHub:
#1 (comment)

from memjs.