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

Posted by qiuchenjian-2 on
URL: http://apache-carbondata-dev-mailing-list-archive.168.s1.nabble.com/GitHub-carbondata-pull-request-1584-CARBONDATA-1827-Added-S3-implementation-and-tp28501p29858.html

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?


---