Move to Hive 2.3.8

[Fokko]

Contains Avro 1.10 :)

[dongjoon-hyun]

It seems that CI fails.

cc @sunchao

[sunchao]

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.

[Fokko]

Thanks for pointing out @sunchao. I've added calcite-core as a dependency as well.

[dongjoon-hyun]

Thank you for updating.

[rdblue]

Why does this need to add Calcite dependencies?

[rdsr]

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

[Fokko]

@sunchao Can you shed some light on why calcite has been removed as a dependency from Hive? I haven't been involved in this discussion.

@rdsr Good catch, I've moved them to testCompile.

[sunchao]

@Fokko it was removed as part of shading Guava- see HIVE-22126 and HIVE-23980.

[rdblue]

Looks good. Thanks for catching the test issue, @rdsr!

Thanks for the update, @Fokko!