westonpace commented on pull request #9474: URL: https://github.com/apache/arrow/pull/9474#issuecomment-777772187
> I've added an owned IOContext to the base FileSystem class. Should we do that on RandomAccessFile or InputStream as well? My gut says no but I could be convinced otherwise. For example, the S3 filesystem (if I understand correctly) would plug the executor into the AWS client configuration and the random access files / etc. would rely on that and not the Executor directly. At best it would just be for convenience right? A way to provide implementors easy access to the context instead of having to take care of passing it around? ---------------------------------------------------------------- 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
