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

[jackylk]

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?


---