> +    public ListenableFuture<BlobMetadata> blobMetadata(String container, 
> String key) {
> +        return async.getBlobMetadata(container, key);
> +    }
> +
> +    @Override
> +    protected boolean deleteAndVerifyContainerGone(String container) {
> +        async.deleteContainer(container);
> +        try {
> +            return !containerExists(container).get();
> +        } catch (InterruptedException e) {
> +            LOGGER.warn("deleteAndVerifyContainerGone operation execution 
> fail", e);
> +            return true;
> +        } catch (ExecutionException e) {
> +            LOGGER.warn("deleteAndVerifyContainerGone operation execution 
> fail", e);
> +            return true;
> +        }

A couple things here:
* You should wait until the `Future` returned by the `deleteContainer`call 
completes before you check if the container still exists.
* Returning `true` here may generate false positives. If you can't determine if 
the container has been deleted, it is better to propagate the exception and let 
the caller handle the error than giving a potential wrong feedback.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/188/files#r7139735

Reply via email to