----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68083/#review206702 -----------------------------------------------------------
Ship it! Ship It! - Velmurugan Periasamy On July 27, 2018, 3:28 p.m., Colm O hEigeartaigh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68083/ > ----------------------------------------------------------- > > (Updated July 27, 2018, 3:28 p.m.) > > > Review request for ranger. > > > Bugs: RANGER-2167 > https://issues.apache.org/jira/browse/RANGER-2167 > > > Repository: ranger > > > Description > ------- > > This task is to upgrade to the Apache parent pom version 20. This > necessitates a PMD upgrade and quite a few fixes are required as a result. > One notable change is to switch the KMS Keystore implementation to use > ConcurrentHashMap instead of HashTable. > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerValiditySchedule.java > 55f787d23 > > agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerValidator.java > ed5aa8d41 > > agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerValidityScheduleValidator.java > f34d124d6 > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerResourceACLs.java > 34098fab6 > > agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java > e16148d4b > > agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerValidityScheduleEvaluator.java > b48ff3b68 > > agents-common/src/test/java/org/apache/ranger/plugin/model/validation/TestRangerValidator.java > f9b342832 > dev-support/ranger-pmd-ruleset.xml f03cda371 > kms/src/main/java/org/apache/hadoop/crypto/key/RangerKeyStore.java > cd5a8c2a8 > > plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java > 262447857 > > plugin-kms/src/main/java/org/apache/ranger/authorization/kms/authorizer/RangerKmsAuthorizer.java > c3d75a14d > > plugin-kms/src/main/java/org/apache/ranger/services/kms/client/KMSClient.java > af0ac71f0 > > plugin-nifi/src/main/java/org/apache/ranger/services/nifi/client/NiFiConnectionMgr.java > 739bef62c > pom.xml 8d7ea131e > > ranger-tools/src/test/java/org/apache/ranger/policyengine/RangerPolicyEnginePerformanceTest.java > 11af0a84d > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > e4449df2e > > security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerKRBAuthenticationFilter.java > b4a3f9355 > > security-admin/src/main/java/org/apache/ranger/security/web/filter/RangerKrbFilter.java > ca0d17eb6 > > security-admin/src/main/java/org/apache/ranger/service/RangerPolicyService.java > a3ff825dd > > security-admin/src/main/java/org/apache/ranger/service/RangerServiceService.java > df3fdb57c > security-admin/src/test/java/org/apache/ranger/biz/TestServiceDBStore.java > b0b011022 > > > Diff: https://reviews.apache.org/r/68083/diff/1/ > > > Testing > ------- > > > Thanks, > > Colm O hEigeartaigh > >