-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68536/
-----------------------------------------------------------
(Updated Aug. 29, 2018, 11:10 a.m.)
Review request for Sqoop, daniel voros, Fero Szabo, Nguyen Truong, and Szabolcs Vasas.
Changes
-------
Simplified validation condition, added poitive path test case
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/2/
Changes: https://reviews.apache.org/r/68536/diff/1-2/
Testing
-------
ant clean test
./gradlew test
Thanks,
Boglarka Egyed
|