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

Posted to dev@sqoop.apache.org by Jarek Cecho <ja...@apache.org> on 2015/09/25 00:52:38 UTC

Re: Review Request 37741: SQOOP-2524 Sqoop2: Add S3 support to HDFS Connector

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37741/
-----------------------------------------------------------

(Updated Sept. 24, 2015, 10:52 p.m.)


Review request for Sqoop.


Changes
-------

Rebased.


Bugs: SQOOP-2524
    https://issues.apache.org/jira/browse/SQOOP-2524


Repository: sqoop-sqoop2


Description
-------

I've fixed few places that needed to be altered in order to make S3 integration work. I've also added a integration test :)


Diffs (updated)
-----

  common/src/main/java/org/apache/sqoop/common/MapContext.java fc722c0 
  common/src/test/java/org/apache/sqoop/common/TestMapContext.java 2a27c0c 
  common/src/test/java/org/apache/sqoop/common/TestMutableMapContext.java db7fa34 
  connector/connector-hdfs/pom.xml 512b54c 
  connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsFromInitializer.java 6c943a8 
  connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsToInitializer.java 5bb0928 
  pom.xml 6e334a7 
  test/pom.xml 3e11f59 
  test/src/main/java/org/apache/sqoop/test/minicluster/TomcatSqoopMiniCluster.java 83f42b6 
  test/src/test/java/org/apache/sqoop/integration/connector/hdfs/S3Test.java PRE-CREATION 

Diff: https://reviews.apache.org/r/37741/diff/


Testing
-------

Integration test is passing.


Thanks,

Jarek Cecho