> @@ -69,12 +69,7 @@ protected HPCloudObjectStorageBlobStore(BlobStoreContext 
> context, BlobUtils blob
>  
>     @Override
>     public boolean createContainerInLocation(Location location, String 
> container, CreateContainerOptions options) {
> -      try {
> -         return createContainerInLocation(location, container);
> -      } finally {
> -         if (options.isPublicRead())
> -            enableAndCache.apply(container);
> -      }
> +      return options.isPublicRead() ? enableAndCache.apply(container) != 
> null : createContainerInLocation(location, container);

> I will clarify this with a comment and rename the enableAndCache method.

Thanks for explaining, Chris! Given that, I think a short comment "enabling CDN 
on the container also creates it if it does not exist" would be fine for me.

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

Reply via email to