----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69063/ -----------------------------------------------------------
Review request for Sqoop, Fero Szabo, Ferenc Szabo, and Szabolcs Vasas. Bugs: SQOOP-3391 https://issues.apache.org/jira/browse/SQOOP-3391 Repository: sqoop-trunk Description ------- Test storing AWS credentials in Hadoop CredentialProvider during import in case of - CredntialProvider with default password - CredntialProvider with password stored in environment variable - CredntialProvider with password file Added test cases for happy and sad paths as well. Added a new test dependency both in Ant and Gradle for setting environment variables in tests easily. Diffs ----- build.gradle 7a0712e3242e31ef2593c34f469f9136cf5dc85d build.xml f3975317140e66c700d85231669ccb2b70367f80 conf/password-file.txt PRE-CREATION conf/wrong-password-file.txt PRE-CREATION gradle.properties 4808ec7d090b9732f9246f21e44bd736adf6efd0 ivy.xml 91157ca74bee3b50269564ddb747638946e45a7e ivy/libraries.properties 2ca95ee99c09fe1aaff6797a6ee0958ac1977663 src/java/org/apache/sqoop/util/password/CredentialProviderHelper.java 1d6481a0697db2fc0ffeb1b012bb143beb615bc0 src/test/org/apache/sqoop/s3/TestS3ImportWithHadoopCredProvider.java PRE-CREATION src/test/org/apache/sqoop/testutil/S3TestUtils.java c9d17bc728d6a229e32c157b56268d6418b3de94 Diff: https://reviews.apache.org/r/69063/diff/1/ Testing ------- ./gradlew test -Ds3.bucket.url=<bucket-url> -Ds3.generator.command=<credential-generator-command> ant clean test -Ds3.bucket.url=<bucket-url> -Ds3.generator.command=<credential-generator-command> Thanks, Boglarka Egyed