-----------------------------------------------------------
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

Reply via email to