-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62522/
-----------------------------------------------------------

Review request for ranger, Alok Lal, Ankita Sinha, Don Bosco Durai, Colm O 
hEigeartaigh, Gautam Borad, Madhan Neethiraj, Ramesh Mani, Selvamohan 
Neethiraj, Velmurugan Periasamy, and Qiang Zhang.


Bugs: RANGER-1804
    https://issues.apache.org/jira/browse/RANGER-1804


Repository: ranger


Description
-------

The exception information is error when judge password for 
RangerKeyStoreProvider class
I think it should throw the password is error instead of throw master key not 
exists.
{code}
String password = conf.get(ENCRYPTION_KEY);
                if(password == null || password.trim().equals("") || 
password.trim().equals("_") || password.trim().equals("crypted")){
                        throw new IOException("Master Key Jceks does not 
exists");
                }
{code}


Diffs
-----

  kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStoreProvider.java 
267fcf08 


Diff: https://reviews.apache.org/r/62522/diff/1/


Testing
-------


Thanks,

wang yuan

Reply via email to