----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68290/#review207109 -----------------------------------------------------------
agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerAccessResource.java Lines 52 (patched) <https://reviews.apache.org/r/68290/#comment290331> Hive specific items should not be brought into common plugin classes. agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerAccessResourceImpl.java Lines 40 (patched) <https://reviews.apache.org/r/68290/#comment290332> Hive specific items should not be brought into common plugin classes. agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerAccessResourceReadOnly.java Lines 74 (patched) <https://reviews.apache.org/r/68290/#comment290333> Hive specific items should not be brought into common plugin classes. agents-common/src/main/java/org/apache/ranger/plugin/policyresourcematcher/RangerDefaultPolicyResourceMatcher.java Line 579 (original), 579 (patched) <https://reviews.apache.org/r/68290/#comment290334> Hive specific items should not be brought into common plugin classes. - Velmurugan Periasamy On Aug. 10, 2018, 11:14 a.m., dhomme wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68290/ > ----------------------------------------------------------- > > (Updated Aug. 10, 2018, 11:14 a.m.) > > > Review request for ranger. > > > Bugs: RANGER-2185 > https://issues.apache.org/jira/browse/RANGER-2185 > > > Repository: ranger > > > Description > ------- > > When a user execute 'show databases;' via hive, ranger hive plugin should > check if the user has any databases to access but not all(*). > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerAccessResource.java > 2ee616a1b > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerAccessResourceImpl.java > 580048627 > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerAccessResourceReadOnly.java > 18bb1f44f > > agents-common/src/main/java/org/apache/ranger/plugin/policyengine/RangerPolicyRepository.java > 23d1efa15 > > agents-common/src/main/java/org/apache/ranger/plugin/policyresourcematcher/RangerDefaultPolicyResourceMatcher.java > 415263eef > > hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java > 3ed79df83 > > > Diff: https://reviews.apache.org/r/68290/diff/3/ > > > Testing > ------- > > Tesetd with local env. > > > Thanks, > > dhomme > >