Skip to content

feat(storage): make GrpcRustClient available as a transport option#6000

Open
joshuatants wants to merge 1 commit into
googleapis:mainfrom
joshuatants:grpc-rust-storage-bidi-alias
Open

feat(storage): make GrpcRustClient available as a transport option#6000
joshuatants wants to merge 1 commit into
googleapis:mainfrom
joshuatants:grpc-rust-storage-bidi-alias

Conversation

@joshuatants

@joshuatants joshuatants commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

For #5991.

@product-auto-label product-auto-label Bot added the api: storage Issues related to the Cloud Storage API. label Jul 8, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces conditional compilation support for an unstable gRPC Rust client implementation (google_cloud_unstable_grpc_rust) in the storage service. It conditionally defines the GrpcClient and GrpcStream types for both bidirectional read and write operations, falling back to the standard gaxi::grpc::Client and tonic::Streaming types when the feature is not enabled. There are no review comments, and I have no feedback to provide.

joshuatants added a commit that referenced this pull request Jul 10, 2026
For #5991.

Introducing a skeleton unblocks working on the changes needed for
Storage for both the hand-written
(#6000) and GAPIC
(changes to be made to the relevant templates in `librarian`) clients.
@joshuatants joshuatants force-pushed the grpc-rust-storage-bidi-alias branch from 849271e to 3277faf Compare July 10, 2026 02:18
@joshuatants joshuatants requested a review from vsharonlynn July 10, 2026 02:23
@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.08%. Comparing base (2e72e31) to head (3277faf).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #6000   +/-   ##
=======================================
  Coverage   97.08%   97.08%           
=======================================
  Files         244      244           
  Lines       61351    61351           
=======================================
+ Hits        59560    59564    +4     
+ Misses       1791     1787    -4     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@joshuatants joshuatants marked this pull request as ready for review July 10, 2026 02:47
@joshuatants joshuatants requested review from a team as code owners July 10, 2026 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants