You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Add support in Storage RPC level for request with selected fields. #193
Storage service allows getting partial blob or gcs object data using selected fields.
We could use that for exists checks and for listing calls + lazy load of the metadata when requested in the functional blobs.
This issue is an optimization only. If we think we want to expose the selected fields feature in the user level we should create a separate issue (and discuses how to make sure one does not override metadata
with partial metadata there).
I am working on this now. I wanted to provide fields selection for blob get and list. My idea was to allow this through a vararg of fields objects/enums. I would also like to encapsulate this in our XXXOption vargargs to avoid having other parameters floating around. This applies quite well to list:
For get we have instead BlobSourceOption that is used in several points across our API. In such points having BlobSourceOption.fields does not always make sense (think of update where selected fields are supported but we also have BlobTargetOption that might be used to specify them).
An alternative could be:
This issue was about adding the selective fields to the RPC (spi) layer rather than adding it to the API layer (so basically it is pure optimization for some operations such as exists).
Do you really think it would be useful to users? If so, what do you feel about creating a separate BlobReadOptions (similar to the BlobWriteOption you already created) to be used with get operation?
Activity
mziccard commentedon Oct 30, 2015
I am working on this now. I wanted to provide fields selection for blob
getandlist. My idea was to allow this through a vararg of fields objects/enums. I would also like to encapsulate this in our XXXOption vargargs to avoid having other parameters floating around. This applies quite well tolist:For
getwe have insteadBlobSourceOptionthat is used in several points across our API. In such points havingBlobSourceOption.fieldsdoes not always make sense (think of update where selected fields are supported but we also haveBlobTargetOptionthat might be used to specify them).An alternative could be:
However, I don't like very much this signature.
Do you have any suggestion?
aozarov commentedon Oct 30, 2015
This issue was about adding the selective fields to the RPC (spi) layer rather than adding it to the API layer (so basically it is pure optimization for some operations such as exists).
Do you really think it would be useful to users? If so, what do you feel about creating a separate
BlobReadOptions(similar to theBlobWriteOptionyou already created) to be used with get operation?aozarov commentedon Nov 30, 2015
I think this issue was fixed, right @mziccard ?
mziccard commentedon Nov 30, 2015
Yes. Closed with #312
chore: include generated samples dir (#193)
chore: include generated samples dir (#193)
feat: Enable REST transport for most of Java and Go clients (#193)
build(deps): update dependency org.codehaus.mojo:build-helper-maven-p…
deps: update dependency com.google.cloud:google-cloud-shared-dependen…
chore(master): release 0.2.0 (#180)
chore(main): release 0.4.0 (googleapis#193)
7 remaining items
chore(main): release 2.3.1 (#194)
chore(main): release 0.4.2 (#194)
build(deps): update dependency org.apache.maven.plugins:maven-deploy-…
chore(main): release 1.1.2-SNAPSHOT (#193)
build(deps): update dependency org.apache.maven.plugins:maven-project…
deps: update dependency google-cloud-storage to v2.5.0 (#193)
chore(main): release 1.3.0 (#172)
build(deps): update dependency org.codehaus.mojo:build-helper-maven-p…
chore: add api-logging to codeowners (#193)
build: add cloud-RAD for Java (#193)
Option to suppress end user credentials warning. (#207)
build(deps): update dependency org.codehaus.mojo:build-helper-maven-p…
Option to suppress end user credentials warning. (#207)
chore(deps): update dependency com.google.cloud:libraries-bom to v9 (#…
chore(deps): update dependency com.google.cloud:libraries-bom to v9 (#…