marton-bod opened a new pull request #1670: URL: https://github.com/apache/iceberg/pull/1670
Currently Hive engine-related Iceberg classes are split between `iceberg-mr` and `iceberg-hive3`. This makes it difficult and potentially confusing for clients to pull them in as a dependency. For example, if a client uses Hive3, they would - a bit counterintuitively - need to pull in both jars instead of just the `hive3` jar, since `mr` does not contain the hive3-compatible object inspector classes and `hive3` only contains those and nothing else. To solve that, we should build the `mr` main source files as well with Hive3 dependencies in the `hive3` module, and just exclude those couple of object inspectors that rely on Hive2 classes. This ensures that in the end we get a nice standalone `hive3` jar that clients can readily use. ---------------------------------------------------------------- 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]
