[GitHub] [arrow] pitrou commented on pull request #7547: ARROW-8950: [C++] Avoid HEAD when possible in S3 filesystem

2020-06-25 Thread GitBox 


pitrou commented on pull request #7547:
URL: https://github.com/apache/arrow/pull/7547#issuecomment-649636318


   Note to self: instead of basing the default `OpenInputStream(const 
FileInfo&)` on `OpenInputStream(const std::string&)`, it would be more logical 
to do the reverse, so that no subclass needs to override both.



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.

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

[GitHub] [arrow] pitrou commented on pull request #7547: ARROW-8950: [C++] Avoid HEAD when possible in S3 filesystem

2020-06-25 Thread GitBox 


pitrou commented on pull request #7547:
URL: https://github.com/apache/arrow/pull/7547#issuecomment-649634658


   @bkietz Would like your input on the dataset changes.



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.

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