----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50465/#review143877 -----------------------------------------------------------
Ship it! Ship It! - Madhan Neethiraj On July 26, 2016, 9:47 p.m., Abhay Kulkarni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50465/ > ----------------------------------------------------------- > > (Updated July 26, 2016, 9:47 p.m.) > > > Review request for ranger. > > > Bugs: RANGER-1124 > https://issues.apache.org/jira/browse/RANGER-1124 > > > Repository: ranger > > > Description > ------- > > Code changes to guard against potential NPEs and other potential run-time > issues; good coding practices. > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/audit/RangerDefaultAuditHandler.java > 66cee76 > > agents-common/src/main/java/org/apache/ranger/plugin/conditionevaluator/RangerTimeOfDayMatcher.java > f65f29e > > agents-common/src/main/java/org/apache/ranger/plugin/model/validation/RangerPolicyValidator.java > a89ec19 > > agents-common/src/main/java/org/apache/ranger/plugin/policyresourcematcher/RangerDefaultPolicyResourceMatcher.java > 6ea194d > > agents-common/src/main/java/org/apache/ranger/plugin/store/file/TagFileStore.java > a20a9f8 > agents-common/src/main/java/org/apache/ranger/plugin/util/SearchFilter.java > cac0181 > > security-admin/src/main/java/org/apache/ranger/common/RangerServicePoliciesCache.java > a68b215 > > security-admin/src/main/java/org/apache/ranger/common/RangerServiceTagsCache.java > 66c1562 > > tagsync/src/main/java/org/apache/ranger/tagsync/sink/tagadmin/TagAdminRESTSink.java > dfc3fdb > > tagsync/src/main/java/org/apache/ranger/tagsync/source/atlas/AtlasNotificationMapper.java > 793ac28 > > Diff: https://reviews.apache.org/r/50465/diff/ > > > Testing > ------- > > Built clean and ran unit tests > > > Thanks, > > Abhay Kulkarni > >
