gaul requested changes on this pull request.

This looks great!

> @@ -192,8 +192,7 @@ public void setContainerAccess(String container, 
> ContainerAccess access) {
    public PageSet<? extends StorageMetadata> list(String container, 
ListContainerOptions options) {
       ListObjectOptions listOptions = 
listContainerOptionsToListObjectOptions.apply(options);
       ListPageWithPrefixes<GoogleCloudStorageObject> gcsList = 
api.getObjectApi().listObjects(container, listOptions);
-      PageSet<? extends StorageMetadata> list = 
objectListToStorageMetadata.apply(gcsList);
-      return options.isDetailed() ? 
fetchBlobMetadataProvider.get().setContainerName(container).apply(list) : list;

Can we now remove `FetchBlobMetadata`?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1184#pullrequestreview-102043881

Reply via email to