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

   # Rationale for this change
    
   Hi team, when reading the code, I found `CompleteMultipart` doesn't populate 
the default headers
   
https://github.com/apache/arrow-rs-object-store/blob/e89f62b6508f6f65873ce6a5017cd0d5d7395184/src/aws/client.rs#L787-L792
   
   While headers are added for other counterpart operations
   PUT: 
https://github.com/apache/arrow-rs-object-store/blob/e89f62b6508f6f65873ce6a5017cd0d5d7395184/src/aws/client.rs#L541-L544
   GET: 
https://github.com/apache/arrow-rs-object-store/blob/e89f62b6508f6f65873ce6a5017cd0d5d7395184/src/aws/client.rs#L872-L875
   
   # What changes are included in this PR?
   
   This PR populate headers to complete multipart upload HTTP request.
   
   # Are there any user-facing changes?
   
   No.


-- 
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