Github user vdiravka commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1111#discussion_r167947402
  
    --- Diff: contrib/storage-hive/hive-exec-shade/pom.xml ---
    @@ -39,23 +39,28 @@
               <groupId>log4j</groupId>
             </exclusion>
             <exclusion>
    -          <groupId>commons-codec</groupId>
    -          <artifactId>commons-codec</artifactId>
    -        </exclusion>
    -        <exclusion>
    -          <artifactId>calcite-avatica</artifactId>
    -          <groupId>org.apache.calcite</groupId>
    +          <groupId>org.json</groupId>
    +          <artifactId>json</artifactId>
             </exclusion>
           </exclusions>
         </dependency>
    +    <dependency>
    +      <groupId>org.apache.parquet</groupId>
    +      <artifactId>parquet-column</artifactId>
    +      <version>${parquet.version}</version>
    --- End diff --
    
    `hive-exec` uses only two parquet dependencies: 
[parquet-column](https://github.com/apache/hive/blob/branch-2.1/ql/pom.xml#L444)
 and 
[parquet-hadoop-bundle](https://github.com/apache/hive/blob/branch-2.1/ql/pom.xml#L109).
    But Drill doesn't use own version of `parquet-hadoop-bundle` and moreover 
Drill version for it is absent in maven repository.
    
    It appears that Hive 2.3.2 uses `parquet-column` 1.8.1 version as well. But 
last Apache Hive master is updated to 1.9.0 version.
    I have added comment about it into `drill-hive-exec-shaded` POM to update 
it in future.


---

Reply via email to