marton-bod edited a comment on pull request #1478: URL: https://github.com/apache/iceberg/pull/1478#issuecomment-695824861
I suppose it might work, but I think it would give us more useful test coverage if the other iceberg subprojects, which `mr` depends on, are built using Hive3/Hadoop3 as well. For instance, most likely if the `mr` code uses Hive3, then the metastore it connects to will use Hive3 as well, so their interoperability should be tested. Let me know what you think. However, just to reiterate, the idea is that for the 'normal' gradle build, all modules will continue to be built using Hive2/Hadoop2 just as before, with no changes. Only when specifying the `-Dhive3` flag to the build, will Hive3/Hadoop3 dependencies be used (and when this flag's enabled, only those modules are included that are ready for the Hive3-build - i.e. Spark and Flink excluded for now, although both Spark3 and Flink could be included at any point if the community agreed, since they already pass all the tests) ---------------------------------------------------------------- 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]
