feat(storage): update request models for appendable upload - #5957
Conversation
There was a problem hiding this comment.
Code Review
This pull request implements bidirectional streaming writes (Bidi Write) for Google Cloud Storage, introducing the AppendableObjectWriter along with request builders OpenAppendableObject and ReopenAppendableObject to support appending data to objects. It includes a background Worker to manage the gRPC stream, a Connector to handle connection handshakes and routing redirects, and comprehensive unit and integration tests. The review feedback highlights two opportunities in connector.rs to improve error handling by replacing .expect() calls with safe error propagation using ? when converting parameters to proto format.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #5957 +/- ##
=======================================
Coverage 96.55% 96.55%
=======================================
Files 257 259 +2
Lines 65173 65173
=======================================
+ Hits 62927 62930 +3
+ Misses 2246 2243 -3 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
7e41183 to
5e066c1
Compare
|
I ran The output is: |
coryan
left a comment
There was a problem hiding this comment.
This is too large for a single PR, please split.
Sorry about this. I have chained PRs, this has changes from #5956. Because my branches are in my fork, I can't diff this branch from the #5956 branch in a PR into this repository. One way I found on how to view only the changes for this PR is to select the commits that don't belong to the previous PR:
Happy to split it up if this still looks too large.
Another way is for us to wait until the previous PR is merged, then I'll rebase this before you take a look. Please let me know if there is a more efficient way to do this... Thank you. |
Yes, that works for me.
Support for stacked PRs (allegedly in the works at GitHub) may help, but I do not think that is available to us yet. Normally we keep stacked PRs as draft. They are useful to give reviewers a sense of the overall change, but too large to review. Reviewing after a rebase is far more practical. |
5e066c1 to
22519c7
Compare
22519c7 to
7419b0b
Compare
7419b0b to
325dfe0
Compare
|
This large PR has been split into 4 PRs:
Thank you. |
This PR comes after PR #5957.

This PR comes after PR #6002 .