-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68536/
-----------------------------------------------------------
(Updated Aug. 30, 2018, 9:59 a.m.)
Review request for Sqoop, daniel voros, Fero Szabo, Nguyen Truong, and Szabolcs
Vasas.
Changes
-------
Changed to startsWith() usage, looking for s3a uri scheme intead of "s3" only
Bugs: SQOOP-3368
https://issues.apache.org/jira/browse/SQOOP-3368
Repository: sqoop-trunk
Description
-------
The current implementation of Sqoop handles HDFS as a default filesystem, i.e.
it creates temporary directories on HDFS in case of incremental append or merge
imports. To make these incremental import use cases work with S3 the user needs
to set the --temporary-rootdir to an S3 location properly.
Diffs (updated)
-----
src/java/org/apache/sqoop/tool/ImportTool.java
139733732d2a28d171568b9118c98a47a3d2fc50
src/test/org/apache/sqoop/tool/TestS3IncrementalImportOptionValidations.java
PRE-CREATION
Diff: https://reviews.apache.org/r/68536/diff/3/
Changes: https://reviews.apache.org/r/68536/diff/2-3/
Testing
-------
ant clean test
./gradlew test -Ds3.bucket.url=<bucket-url>
-Ds3.generator.command=<credential-generator-command>
Thanks,
Boglarka Egyed
