Lost error message and exception #5

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@raphael-wb

When Acquire fails, it may return meaningful error message and exception.

Within ExecuteOperation, "readResult.Combine(result);" overwrites this result with the values of "readResult", which in this block is always blank.

Dropping it just gave me back the error I was looking for.

I've reported the bug in the tracking tool: http://www.couchbase.com/issues/browse/NCBC-281

Raphaël Bartement
Fix: When Acquire fails, ExecuteOperation overwrite its result with a…
… blank one. Error message and exception are lost.

@ghost ghost assigned jeffrymorris Oct 3, 2013

ns-codereview pushed a commit that referenced this pull request Dec 3, 2013

NCBC-310 Amendment #5 - operation timeout
This commit adds timeouts to operations where there is potential for
the client to hang indefinitly.

Change-Id: Id50b6a41167b12c0d8b22d2c939a3ee8846ddf30
Reviewed-on: http://review.couchbase.org/30777
Reviewed-by: Jeffry Morris <jeffrymorris@gmail.com>
Tested-by: Jeffry Morris <jeffrymorris@gmail.com>
@jeffrymorris

This comment has been minimized.

Show comment Hide comment
@jeffrymorris

jeffrymorris Jan 30, 2015

Member

@raphael-wb

Can you verify if this has already been fixed? If so can you close this PR?

Thanks!

-Jeff

Member

jeffrymorris commented Jan 30, 2015

@raphael-wb

Can you verify if this has already been fixed? If so can you close this PR?

Thanks!

-Jeff

@brett19 brett19 closed this Sep 17, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment