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

Posted to github@arrow.apache.org by "raulcd (via GitHub)" <gi...@apache.org> on 2026/06/04 07:24:34 UTC

Re: [PR] GH-46369: [C++] Add key-value pairs to the FileSystemFactory interface [arrow]

raulcd commented on PR #50044:
URL: https://github.com/apache/arrow/pull/50044#issuecomment-4619945278

   @pitrou any thoughts about the scope for this PR, as shared above:
   > About the scope of this PR, would you push this plumbing PR in isolation (in which case I would add a couple of tests to validate options are passed), or should I add some S3 specific options as part of this PR?
   
   > For the scope of the S3 options (on this PR or a follow up one), would you mimic all S3Options as part of the new k/v pair or only a small subset, example initially only region and credentials?


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