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 2021/08/16 17:23:52 UTC

[GitHub] [arrow-datafusion] ehenry2 edited a comment on pull request #811: Add support for reading remote storage systems

ehenry2 edited a comment on pull request #811:
URL: https://github.com/apache/arrow-datafusion/pull/811#issuecomment-899683040


   This is an awesome PR...one question for clarification...should the object reader trait have all its functions return Result? Remote storage systems have the wonderful feature of potentially having IO errors at any moment (as the request is being made over the wire) so even innocuous functions like length() have the potential to fail. I'm viewing this from the lens of S3, as I'd assuming a length() request would involve a head_object request or such to the s3 api, or should that info be pre-populated when the ObjectReader is instantiated? 


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