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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
