pvary commented on pull request #1505: URL: https://github.com/apache/iceberg/pull/1505#issuecomment-698853877
> 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. Implemented the config property + hive-site.xml solution. @rdblue: Could you please check that this is the same solution you were thinking about? Also please double check the dependency changes I have added. Thanks, Peter ---------------------------------------------------------------- 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]
