[GitHub] [iceberg] jackye1995 commented on pull request #1618: refactor util to dynamically load location provider and file io

[GitBox <gi...@apache.org>]

jackye1995 commented on pull request #1618:
URL: https://github.com/apache/iceberg/pull/1618#issuecomment-719117071


   @rdblue So in the latest version, I have added nullable field `fileIOImpl` as a part of constructor for `HadoopTableOperations` and `HiveTableOperations`. The `HadoopCatalog`, `HadoopTables` and `HiveCatalog` uses Hadoop configuration to decide the value, which should be fine because they are already using the configuration to determine some other important configs. Other catalog implementations can use any way to pass in the value.


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org