You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jclouds.apache.org by ga...@apache.org on 2014/08/22 01:19:57 UTC

git commit: Throw ContainerNotFoundException from removeBlob

Repository: jclouds
Updated Branches:
  refs/heads/master 80a443003 -> 60e376399


Throw ContainerNotFoundException from removeBlob

This makes removeBlob consistent with all other user-facing methods.


Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo
Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/60e37639
Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/60e37639
Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/60e37639

Branch: refs/heads/master
Commit: 60e37639998543d14309791b5f2ecd2a9cf0ec3e
Parents: 80a4430
Author: Andrew Gaul <ga...@apache.org>
Authored: Thu Aug 21 16:18:12 2014 -0700
Committer: Andrew Gaul <ga...@apache.org>
Committed: Thu Aug 21 16:18:12 2014 -0700

----------------------------------------------------------------------
 .../src/main/java/org/jclouds/blobstore/LocalAsyncBlobStore.java  | 3 +++
 1 file changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/jclouds/blob/60e37639/blobstore/src/main/java/org/jclouds/blobstore/LocalAsyncBlobStore.java
----------------------------------------------------------------------
diff --git a/blobstore/src/main/java/org/jclouds/blobstore/LocalAsyncBlobStore.java b/blobstore/src/main/java/org/jclouds/blobstore/LocalAsyncBlobStore.java
index 71c2c4e..42e4124 100644
--- a/blobstore/src/main/java/org/jclouds/blobstore/LocalAsyncBlobStore.java
+++ b/blobstore/src/main/java/org/jclouds/blobstore/LocalAsyncBlobStore.java
@@ -229,6 +229,9 @@ public class LocalAsyncBlobStore extends BaseAsyncBlobStore {
     */
    @Override
    public ListenableFuture<Void> removeBlob(final String container, final String key) {
+      if (!storageStrategy.containerExists(container)) {
+         return Futures.immediateFailedFuture(cnfe(container));
+      }
       storageStrategy.removeBlob(container, key);
       return immediateFuture(null);
    }