Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(GCS+gRPC): synthetic object metadata links #7563

Conversation

coryan
Copy link
Contributor

@coryan coryan commented Nov 4, 2021

The JSON API provides links to the media and metadata contents of each
object as part of the object metadata. The gRPC API does not provide
such links. While not strictly necessary, I think it is nice to create
synthetic versions of these links.


This change is Reviewable

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Nov 4, 2021
@google-cloud-cpp-bot
Copy link
Collaborator

Google Cloud Build Logs
For commit: 63bfff0996cd09e382deb70691313f1043f1c156

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

@codecov
Copy link

codecov bot commented Nov 4, 2021

Codecov Report

Merging #7563 (4718572) into main (656932b) will decrease coverage by 0.00%.
The diff coverage is 88.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #7563      +/-   ##
==========================================
- Coverage   93.31%   93.31%   -0.01%     
==========================================
  Files        1381     1381              
  Lines      119588   119606      +18     
==========================================
+ Hits       111598   111613      +15     
- Misses       7990     7993       +3     
Impacted Files Coverage Δ
...torage/tests/object_basic_crud_integration_test.cc 80.36% <0.00%> (-1.00%) ⬇️
google/cloud/storage/internal/grpc_client.cc 92.23% <94.44%> (+0.18%) ⬆️
google/cloud/storage/internal/grpc_client.h 100.00% <100.00%> (ø)
google/cloud/storage/internal/grpc_client_test.cc 100.00% <100.00%> (ø)
.../storage/internal/grpc_resumable_upload_session.cc 100.00% <100.00%> (ø)
google/cloud/storage/object_metadata.cc 99.34% <100.00%> (+<0.01%) ⬆️
google/cloud/pubsub/samples/samples.cc 92.02% <0.00%> (-0.08%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 656932b...4718572. Read the comment docs.

@coryan coryan marked this pull request as ready for review November 4, 2021 14:48
@coryan coryan requested a review from a team as a code owner November 4, 2021 14:48
The JSON API provides links to the media and metadata contents of each
object as part of the object metadata. The gRPC API does not provide
such links. While not strictly necessary, I think it is nice to create
synthetic versions of these links.
@coryan coryan force-pushed the feat-GCS+gRPC-create-synthetic-object-metadata-links branch from 63bfff0 to 4718572 Compare November 5, 2021 16:44
@google-cloud-cpp-bot
Copy link
Collaborator

Google Cloud Build Logs
For commit: 471857273a5444dac7a97cd0668c09c395a517c9

ℹ️ NOTE: Kokoro logs are linked from "Details" below.

@coryan coryan merged commit b3403ba into googleapis:main Nov 5, 2021
@coryan coryan deleted the feat-GCS+gRPC-create-synthetic-object-metadata-links branch November 5, 2021 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants