rdblue commented on pull request #1505: URL: https://github.com/apache/iceberg/pull/1505#issuecomment-698533716
I don't think the reflection option would have any benefit because Hive would still need `ReflectionStorageHandler` in the classpath. If we can guarantee that something is in the classpath, then we should use `HiveIcebergStorageHandler` so everything works. Then we have the choice between whether we break Hive or not. I think a config property is the right way to go so that it is up to the user or administrator whether to expect Iceberg in the classpath. I think maybe a Hadoop Configuration option in hive-site.xml is a good idea. ---------------------------------------------------------------- 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]
