jbonofre commented on PR #9884: URL: https://github.com/apache/iceberg/pull/9884#issuecomment-1986753344
@danielcweeks I know for FileIO, but I don't understand why we have `s3` and `s3a` in `ResolvingFileIO`, actually why do we have `ResolvingFileIO` ? My understanding is that `ResolvingFileIO` gives a change to "bypass" Hadoop/FileIO and be able to have a "core" Iceberg implementation (without necessarily extending HadoopFileIO). So, this change is exactly for this: let the user to define its own Iceberg custom implementation without necessarily going on HadoopFileIO extend. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
