----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73634/#review223580 -----------------------------------------------------------
kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSMetricUtil.java Line 35 (original), 35 (patched) <https://reviews.apache.org/r/73634/#comment312661> Please consider removing un-necessary spaces. kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSMetricUtil.java Line 123 (original), 124 (patched) <https://reviews.apache.org/r/73634/#comment312662> Please consider removing un-necessary spaces. kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSMetricUtil.java Line 137 (original), 137 (patched) <https://reviews.apache.org/r/73634/#comment312663> Please consider removing un-necessary spaces. - Kishor Gollapalliwar On Oct. 7, 2021, 5:07 a.m., Mateen Mansoori wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73634/ > ----------------------------------------------------------- > > (Updated Oct. 7, 2021, 5:07 a.m.) > > > Review request for ranger, Abhishek Kumar, Dhaval Shah, Abhay Kulkarni, Mehul > Parikh, Ramesh Mani, and Velmurugan Periasamy. > > > Bugs: RANGER-3463 > https://issues.apache.org/jira/browse/RANGER-3463 > > > Repository: ranger > > > Description > ------- > > Avoid use of System.out.println in KMSMetricUtil.java at line numbers 74, 79, > 98, 104. > > > Diffs > ----- > > > kms/src/main/java/org/apache/hadoop/crypto/key/kms/server/KMSMetricUtil.java > 925d2c76f > > > Diff: https://reviews.apache.org/r/73634/diff/1/ > > > Testing > ------- > > Tested on local VM. > > > Thanks, > > Mateen Mansoori > >
