You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "RanaPriyansh (via GitHub)" <gi...@apache.org> on 2026/09/22 08:48:14 UTC

[PR] GCP: Request uncompressed credential and signing responses [arrow-rs-object-store]

RanaPriyansh opened a new pull request, #860:
URL: https://github.com/apache/arrow-rs-object-store/pull/860

   # Which issue does this PR close?
   
   Closes #783.
   
   # What changes are included in this PR?
   
   GCP credential and signing requests now set `Accept-Encoding: identity`, so a default gzip header does not break response parsing. Object requests retain their configured headers and compressed bytes.
   
   # Are there any user-facing changes?
   
   GCP credentials and remote signing work with a gzip default header. Public APIs are unchanged.
   
   Validation: GCP library tests, all-feature library tests, the custom-transport regression, strict Clippy checks, and formatting pass with Rust 1.97.1. The response-parsing regressions fail on the unpatched source. Real cloud integration was not run.
   
   AI disclosure: OpenAI Codex generated the source changes, tests, and this description.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GCP: Request uncompressed credential and signing responses [arrow-rs-object-store]

Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on PR #860:
URL: https://github.com/apache/arrow-rs-object-store/pull/860#issuecomment-6224941571

   Marking as draft per https://github.com/apache/arrow-rs-object-store/pull/873#issuecomment-6224909416 so i can find the PRs that need attention. Thank you 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GCP: Request uncompressed credential and signing responses [arrow-rs-object-store]

Posted by "alamb (via GitHub) via github" <gi...@arrow.apache.org>.
alamb closed pull request #860: GCP: Request uncompressed credential and signing responses
URL: https://github.com/apache/arrow-rs-object-store/pull/860


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] GCP: Request uncompressed credential and signing responses [arrow-rs-object-store]

Posted by "alamb (via GitHub) via github" <gi...@arrow.apache.org>.
alamb commented on PR #860:
URL: https://github.com/apache/arrow-rs-object-store/pull/860#issuecomment-6558317462

   Thank you for the contribution. The change may well be correct, but I am closing this PR under the arrow-rs guidance on AI generated submissions: https://github.com/apache/arrow-rs/blob/main/CONTRIBUTING.md#ai-generated-submissions
   
   Maintainer review time is the scarcest resource in this project. This account has opened more than 80 PRs across many repositories since October 1, and currently has 9 PRs open in object_store totaling roughly 3,900 added lines for maintainers to review:
   
   - #873 Test external CryptoProvider integration through public S3 APIs (+481/-0)
   - #869 Test downstream HTTP and crypto configurations (+671/-41)
   - #867 fix: validate Azure delegation key coverage (+544/-89)
   - #866 fix: honor fsync for local deletes (+175/-16)
   - #861 fix: honor GCS skip_signature during uploads (+85/-2)
   - #860 GCP: Request uncompressed credential and signing responses (+904/-3)
   - #859 test: cover local filesystem put and delete errors (+110/-0)
   - #858 fix: accept complete HTTP responses for full ranges (+589/-24)
   - #857 fix: preserve typed errors for S3 and GCS list requests (+348/-2)
   
   That volume is far more than we can review. As discussed in #873, we will focus on #861 and I am closing the other draft PRs.
   
   If one of these changes matters to you after #861 is resolved, please open a focused PR with a short description in your own words, one you have reviewed and understand, and we'd be happy to help with review.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org