[GitHub] [incubator-iceberg] massdosage commented on issue #934: We should publish a shaded version of problematic dependencies to use in Iceberg

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

massdosage commented on issue #934:
URL: https://github.com/apache/incubator-iceberg/issues/934#issuecomment-616439191


   Yes, `iceberg-hive` depends on `iceberg-core`, `iceberg-api`, `iceberg-common` and `iceberg-mr` so all of these jar files will need to go on Hive's classpath when you use any of the classes inside iceberg-hive. Due to Guava version conflicts with the version of Guava used by Hive all usages of Guava inside Iceberg's classes need to be shaded and reolcated otherwise there are runtime version conflicts which prevent being able to integrate Iceberg and Hive. I would imagine there would be similar issues with anyone using any Iceberg modules via an API and also having their own version of Guava so would recommend applying this across the whole project if possible.


----------------------------------------------------------------
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