You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "dentiny (via GitHub)" <gi...@apache.org> on 2026/04/09 22:33:53 UTC

[PR] fix(aws): populate default header for complete mutipart request [arrow-rs-object-store]

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: github-unsubscribe@arrow.apache.org

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

Re: [PR] fix(aws): populate default header for complete mutipart request [arrow-rs-object-store]

Posted by "crepererum (via GitHub)" <gi...@apache.org>.

crepererum merged PR #688:
URL: https://github.com/apache/arrow-rs-object-store/pull/688


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