----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58921/ -----------------------------------------------------------
(Updated May 4, 2017, 6 p.m.) Review request for ranger, Gautam Borad, Madhan Neethiraj, and Sailaja Polavarapu. Changes ------- edited diff to address review comments. Repository: ranger Description ------- The default communication between Ranger admin and Ranger usersync or Ranger kms is SSL enabled. When there is an error during the hadoop keystore creation, there is no exception thrown but there was a non-zeor return code returned. However, ranger did not check/handle this error return code and continued, which has caused Ranger usersync to assume the communication to be normal and continue. With this fix, when there is an error during the keystore creation, an error message is logged and the process will exit. Diffs (updated) ----- credentialbuilder/src/main/java/org/apache/ranger/credentialapi/buildks.java a7e433f Diff: https://reviews.apache.org/r/58921/diff/2/ Changes: https://reviews.apache.org/r/58921/diff/1-2/ Testing ------- Manual unit testing & Automated tests. Thanks, Madhavi Amirneni
