-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64034/
-----------------------------------------------------------
Review request for Sqoop.
Bugs: SQOOP-3257
https://issues.apache.org/jira/browse/SQOOP-3257
Repository: sqoop-trunk
Description
-------
If sqoop.metastore.client.record.password property is set to true then Sqoop
stores the database password in the metastore. The problem is that in this case
the password can be logged by a couple of classes which needs to be avoided.
Diffs
-----
src/java/org/apache/sqoop/SqoopJobDataPublisher.java
93be02fa00aab091707b42b32c8a638a881bdf50
src/java/org/apache/sqoop/SqoopOptions.java
587d4e1dd4012692ca7c742bcd518f55c10d1f8f
src/java/org/apache/sqoop/metastore/GenericJobStorage.java
9e1b18b6dad08991e11e41435bf62ea87b534f2f
src/java/org/apache/sqoop/metastore/PasswordRedactor.java PRE-CREATION
src/java/org/apache/sqoop/tool/JobTool.java
dbe89349c17ca86c3985c279cb3fe150e3bcab0e
src/test/org/apache/sqoop/credentials/TestPassingSecurePassword.java
bd911f224496688bae811767c1b559f88965a571
src/test/org/apache/sqoop/metastore/PasswordRedactorTest.java PRE-CREATION
Diff: https://reviews.apache.org/r/64034/diff/1/
Testing
-------
Unit tests and third party tests are executed.
Thanks,
Szabolcs Vasas
