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

    https://github.com/apache/drill/pull/1111#discussion_r166672700
  
    --- 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>
    --- End diff --
    
    The master branch of Hive doesn't have org.json.
    But Hive2.1 includes this:
    https://github.com/apache/hive/blob/branch-2.1/pom.xml#L608
    https://github.com/apache/hive/blob/branch-2.1/pom.xml#L608
    and so on...


---

Reply via email to