search

GoogleCloudPlatform / appengine-gcs-client

App Engine-Cloud Storage custom client library
Apache License 2.0
124 stars 112 forks source link

Remove listing of Blobstore BlobInfo #38

Closed ajkannan closed 7 years ago

ajkannan commented 8 years ago

Reverts the listing changes made in #36. Ideally, we'd return both Blobstore GCS blob metadata and appengine-gcs-client metadata from calls to "list" on local runs. However, the complexities of handling that don't seem worth it, considering that the Blobstore API is deprecated and this API is no longer preferred (see gcloud-java). Though #36 partially supporting listing, there could be unexpected behavior where depending on whether or not you happen to upload any blobs using appengine-gcs-client, you may or may not see your GCS objects uploaded via the Blobstore API.

ajkannan commented 8 years ago

/cc @BrandonY @aozarov

aozarov commented 8 years ago

Looks good to me.

ajkannan commented 7 years ago

This should probably still go in, right @BrandonY ?

BrandonY commented 7 years ago

Yes, this fell off my plate, sorry.