Couchbase exception returns incorrect status

.NET SDK
1

Hello.

When I get the following error:
“The operation 60730/MyKey timed out after 00:00:02.5146141. It was retried 0 times using Couchbase.Core.Retry.BestEffortRetryStrategy”

And then I try to extract the error code from the exception:
((dynamic)ex.Context).Status.ToString();

I get “Success”

2

Can you please post the code?
Thanks.

3

@alon.schachter

Based on the fact that there was 0 retries, I am betting that the operation never made to the network. The default for ResponseStatus is “Success” or 0; my guess it never was set and the default was returned. I don’t believe that is correct behavior but make sense in a way (see NCBC-3266).

I would enable logging and/or use the SDK Dr to help identify the cause of the timeout. The exception should have a ErrorContext field which may also help isolating the cause of the timeout

Jeff

4

I know the cause.
The cause was I took down the service to test what happens. :smiley:

5

Looking at the RetryOrchestrator code, it simply isn’t setting the Status on the KeyValueErrorContext for timeouts. This is somewhat expected because Status is typically the value from the response from the server and there is no response in this case.

However, it does seem confusing. There is a Status value in the enumeration OperationTimeout, which is in the “client level failures” group. Unless this is against the SDK standards for some reason, I think we should set the Status to OperationTimeout in RetryOrchestrator when we throw.

6

https://issues.couchbase.com/browse/NCBC-3266 has been opened for this issue.