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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/04/15 05:06:49 UTC

[GitHub] [arrow] kou commented on pull request #12895: ARROW-16043: [C++][Filesystem][S3] Add missing empty content for creating directory

kou commented on PR #12895:
URL: https://github.com/apache/arrow/pull/12895#issuecomment-1099852467

   Sorry. I describe this problem.
   
   Recent AWS SDK for C++ adds checksum automatically by  
   https://github.com/aws/aws-sdk-cpp/commit/01e61b3137f90509e71765ee7e3b4a39e2e8de91 . The checksum is calculated by request body. If request body is `nullptr`, this problem is occurred.
   
   Each request type knows whether checksum is needed or not. We use `PutObjectRequest` for creating a directory. And it requires `md5` checksum by default by https://github.com/aws/aws-sdk-cpp/commit/24b3cc68451c80a96fbbe133921273b07151d453 .
   
   > the [docs](https://docs.aws.amazon.com/AmazonS3/latest/API/API_PutObject.html) do say this should be optional, so I wonder why this became required in the SDK.
   
   Could you quote the sentence for it? I couldn't find it in the docs...
   
   We may report this as a bug to ASW SDK for C++.


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