----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70456/#review214608 -----------------------------------------------------------
agents-common/src/main/resources/service-defs/ranger-servicedef-hive.json Lines 182 (patched) <https://reviews.apache.org/r/70456/#comment300780> Please make the itemId:14. Couple of JIRAs has additional permissionsto Service Def https://issues.apache.org/jira/browse/RANGER-2389 https://issues.apache.org/jira/browse/RANGER-2391 and it using 11,12,and 13 numbers. So I would suggest to use 14 for this change. - Ramesh Mani On April 11, 2019, 6:47 p.m., Austin Nobis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70456/ > ----------------------------------------------------------- > > (Updated April 11, 2019, 6:47 p.m.) > > > Review request for ranger. > > > Bugs: RANGER-2374 > https://issues.apache.org/jira/browse/RANGER-2374 > > > Repository: ranger > > > Description > ------- > > Created a patch file for the Hive Service Definition to add the Refresh > access type. This is to support sharing a service definition between Apache > Hive and Apache Impala. > > > Diffs > ----- > > agents-common/src/main/resources/service-defs/ranger-servicedef-hive.json > b0f877a01 > security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql > 2eaffccac > security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql > 5ca1e3667 > security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql > 51f4999f7 > > security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql > 5f64a2dba > security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql > fc21cec3e > > security-admin/src/main/java/org/apache/ranger/patch/PatchForHiveServiceDefUpdate_J10027.java > PRE-CREATION > > > Diff: https://reviews.apache.org/r/70456/diff/5/ > > > Testing > ------- > > > Thanks, > > Austin Nobis > >
