> On May 16, 2014, 5:59 a.m., Prasad Mujumdar wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/hadoop/hive/ql/exec/SentryGrantRevokeTask.java, > > line 405 > > <https://reviews.apache.org/r/21496/diff/1/?file=582694#file582694line405> > > > > Does Hive show "none" for non-existent values ? (Just in case if you > > happen to have column or user called 'none' then it will be a problem :) ) > >
Thats true I changed it to null for strings and false for boolean (grant option). > On May 16, 2014, 5:59 a.m., Prasad Mujumdar wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/hadoop/hive/ql/exec/SentryGrantRevokeTask.java, > > line 407 > > <https://reviews.apache.org/r/21496/diff/1/?file=582694#file582694line407> > > > > Is the principal name and type not available ? Yeah, this principle type will always be Role, as we only support role to privilege mapping. - Sravya ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21496/#review43200 ----------------------------------------------------------- On May 15, 2014, 6:58 p.m., Sravya Tirukkovalur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21496/ > ----------------------------------------------------------- > > (Updated May 15, 2014, 6:58 p.m.) > > > Review request for sentry and Prasad Mujumdar. > > > Bugs: SENTRY-150 > https://issues.apache.org/jira/browse/SENTRY-150 > > > Repository: sentry > > > Description > ------- > > Support following commands in hive binding using Sentry service and the > expected output format: > SHOW ROLES > | role | > > SHOW GRANT ROLE roleName > | database | table | partition | column | principal_name | > principal_type | privilege | grant_option | grant_time | grantor | > > SHOW ROLE GRANT GROUP groupName > | role | grant_option | grant_time | grantor | > > > Diffs > ----- > > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/hadoop/hive/ql/exec/SentryGrantRevokeTask.java > 2776eaec97dee25c15a526cd0d70b6a637993b4f > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/SentryHiveAuthorizationTaskFactoryImpl.java > 2e7a9d9043ae0a31d7d9c5f57dac92209feea24e > sentry-tests/sentry-tests-hive/pom.xml > 0005a534876320ff0aededf56a45833e0a6a1d41 > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestDatabaseProvider.java > 0e72a8b07ce9ee033c54428955c8f671a1bbe8de > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/AbstractHiveServer.java > be6eafc226b122345523ccbcf2434f46264edffb > > Diff: https://reviews.apache.org/r/21496/diff/ > > > Testing > ------- > > Added end to end tests which exercise the above commands. > > > Thanks, > > Sravya Tirukkovalur > >
