rdsr commented on pull request #2110: URL: https://github.com/apache/iceberg/pull/2110#issuecomment-770327766
> Great to see this! I think you will need to specify version for calcite-core as well since it is excluded from hive-exec. > > Note Hive 2.3.8 still uses Avro 1.8.2, although it should be compatible with Avro 1.10. Seems like calcite-druid is only required during `iceberg-mr` testing [calcite-core is already include in test scope]. Instead of adding at compileOnly, I think we should only add calcite druid as a test dep under `iceberg-mr` only ---------------------------------------------------------------- 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]
