Github user vdiravka commented on a diff in the pull request:
https://github.com/apache/drill/pull/1111#discussion_r167945192
--- Diff: common/pom.xml ---
@@ -45,6 +45,7 @@
<dependency>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite-core</artifactId>
+ <version>${calcite.version}</version>
--- End diff --
I have returned calcite-core version into `DependencyManagement` block.
Drill Calcite version libraries are included into "drill-hive-exec-shaded"
module.
I will tell you why it works for now:
When user submits query in Drill via Hive plugin, the query is validated
and planned via Drill Calcite. So Hive Calcite isn't necessary for it.
Hive Calcite is used only in the process of Drill unit testing, where a lot
of Hive specific queries are performed to setup Hive store for testing. But
Drill Calcite and Avatica versions have conflicts with Hive old Calcite and
Avatica versions. That's why I have disabled Calcite cost based optimizator
`conf.set(ConfVars.HIVE_CBO_ENABLED.varname, "false");`. We can enable it again
once Hive will leverage the newest Calcite version. The comment about it is
added into `drill-hive-exec-shaded` POM.
---
