You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@carbondata.apache.org by jackylk <gi...@git.apache.org> on 2017/12/07 08:27:46 UTC

[GitHub] carbondata pull request #1584: [CARBONDATA-1827] Added S3 Implementation

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

    https://github.com/apache/carbondata/pull/1584#discussion_r155455752
  
    --- Diff: core/pom.xml ---
    @@ -94,6 +99,20 @@
             </exclusion>
           </exclusions>
         </dependency>
    +
    +    <!-- https://mvnrepository.com/artifact/org.apache.hadoop/hadoop-aws -->
    +    <dependency>
    +      <groupId>org.apache.hadoop</groupId>
    --- End diff --
    
    Why is this required? Core should not dependent on hadoop, right?


---