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

Posted to github@arrow.apache.org by "universalmind303 (via GitHub)" <gi...@apache.org> on 2023/09/18 22:56:27 UTC

[GitHub] [arrow-rs] universalmind303 commented on pull request #4834: fix: object store http header last modified

universalmind303 commented on PR #4834:
URL: https://github.com/apache/arrow-rs/pull/4834#issuecomment-1724573537

   > However, as written this PR will loosen the requirements for things like S3, etc... which I'm not sure is a good idea
   
   That makes sense. I just pushed up a refactor that changes `header_meta` to take in a new config struct that allows users to specify which fields are required. (last_modified, etag) being the two that are optional. 
   
   This should leave the other object stores like S3 unaffected. 


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