alamb commented on PR #7301:
URL: https://github.com/apache/arrow-rs/pull/7301#issuecomment-2734730623

   > I did try the same test with this branch, and it also works fine. So I'll 
leave it up to you if you think it's worth merging. In terms of unit tests, is 
there somewhere where we are already unit testing multipart uploads? It seems 
difficult to test without refactoring s.t. we generate and execute the request 
in separate methods.
   
   I think s3 has multipart upload tests -- but we would have to dig around to 
be sure
   
   I'll plan to merge this PR in over the next few days unless anyone else has 
an objection
   
   (BTW @jkosh44  I wonder if you need to make large multi-part requests -- 
like actually try to upload 10 parts or something)


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