Skip to content

Loading…

Don't swallow errors in ExecuteRemove #7

Merged
merged 1 commit into from

2 participants

@jzablocki

No description provided.

@jzablocki

Bubbling up StatusCode in ExecuteRemove

@jzablocki jzablocki merged commit 4388f4c into couchbase:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 30, 2013
  1. @johnzablocki
Showing with 1 addition and 2 deletions.
  1. +1 −2 Enyim.Caching/MemcachedClient.Results.cs
View
3 Enyim.Caching/MemcachedClient.Results.cs
@@ -458,8 +458,7 @@ public IRemoveOperationResult ExecuteRemove(string key)
}
else
{
- result.InnerResult = commandResult;
- result.Fail("Failed to remove item, see InnerResult or StatusCode for details");
+ commandResult.Combine(result);
}
return result;
Something went wrong with that request. Please try again.