ehenry2 commented on pull request #811: URL: https://github.com/apache/arrow-datafusion/pull/811#issuecomment-899683040
This is an awesome PR...one question though...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-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
