----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72969/#review222247 -----------------------------------------------------------
agents-common/src/main/java/org/apache/ranger/authorization/utils/JsonUtils.java Lines 128 (patched) <https://reviews.apache.org/r/72969/#comment311303> We are returning null. I assume we are handling it where it is been called. Should we just throw the exception and let it get handle where it is called? agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java Lines 907 (patched) <https://reviews.apache.org/r/72969/#comment311304> Should this be accessTime and not accessType? agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAuditPolicyEvaluator.java Lines 74 (patched) <https://reviews.apache.org/r/72969/#comment311305> Let's make sure ++ doesn't mess the passed value. It might be better to increment in a new statement to be safe - Don Bosco Durai On Nov. 30, 2020, 3:42 a.m., Ramesh Mani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72969/ > ----------------------------------------------------------- > > (Updated Nov. 30, 2020, 3:42 a.m.) > > > Review request for ranger, Abhay Kulkarni, Madhan Neethiraj, and Velmurugan > Periasamy. > > > Bugs: RANGER-3000 > https://issues.apache.org/jira/browse/RANGER-3000 > > > Repository: ranger > > > Description > ------- > > RANGER-3000:Audit-filter feature implementation to help reduce volume of > audit logs generated > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/authorization/utils/JsonUtils.java > 994d3944d > agents-common/src/main/java/org/apache/ranger/plugin/model/AuditFilter.java > PRE-CREATION > > agents-common/src/main/java/org/apache/ranger/plugin/model/RangerPolicy.java > 04c6e75c4 > > agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerServiceDefHelper.java > 0d9a346d4 > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerAccessResult.java > c54ef1704 > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngine.java > 4e41adcea > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyEngineImpl.java > b66d5a1ce > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java > 169ed0f5d > > agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerAuditPolicyEvaluator.java > PRE-CREATION > > agents-common/src/main/java/org/apache/ranger/plugin/policyevaluator/RangerDefaultPolicyEvaluator.java > 07fb63872 > > agents-common/src/main/java/org/apache/ranger/plugin/policyresourcematcher/RangerDefaultPolicyResourceMatcher.java > 979488181 > > agents-common/src/main/java/org/apache/ranger/plugin/service/RangerBasePlugin.java > 2d9bc7382 > > agents-common/src/main/java/org/apache/ranger/plugin/util/ServicePolicies.java > 6ab068f6f > > agents-common/src/test/java/org/apache/ranger/plugin/policyengine/TestPolicyEngine.java > 63fccd0b3 > > agents-common/src/test/resources/policyengine/test_policyengine_audit_filter_hdfs.json > PRE-CREATION > > agents-common/src/test/resources/policyengine/test_policyengine_audit_filter_hive.json > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/rest/ServiceREST.java > 520258715 > > > Diff: https://reviews.apache.org/r/72969/diff/5/ > > > Testing > ------- > > - Testing done in local VM. > > > Thanks, > > Ramesh Mani > >