phillipleblanc opened a new pull request, #6843:
URL: https://github.com/apache/arrow-rs/pull/6843

   # Which issue does this PR close?
   
   Closes #6842
   
   # Rationale for this change
    
   Fixes an issue where enabling a non-default feature (`gzip`) for `reqwest` 
would cause `object_store` to stop working.
   
   # What changes are included in this PR?
   
   Call the [`no_gzip`] method on the reqest ClientBuilder to ensure that even 
if the `gzip` feature is enabled, the `object_store` client will not use the 
transparent decompression logic.
   
   [`no_gzip`]: 
https://docs.rs/reqwest/latest/reqwest/struct.ClientBuilder.html#method.no_gzip
   
   I considered making this an option instead of always setting it, but since 
this is such a frustrating footgun to encounter and debug, I think its better 
to always set it unless there is a compelling reason not to.
   
   # Are there any user-facing changes?
   
   My understanding is that most/all users interacting with object stores do 
not want the `gzip` compression logic (since none of the major cloud object 
store providers support it), so this change should not be breaking.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to