----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22105/ -----------------------------------------------------------
(Updated June 2, 2014, 6:09 p.m.) Review request for sentry, Arun Suresh and Prasad Mujumdar. Changes ------- Updated the patch with review comments. Also "grant all on server" privilege was being shown as empty value in database column, fixed it to instead show "*". Bugs: SENTRY-215 https://issues.apache.org/jira/browse/SENTRY-215 Repository: sentry Description ------- Support SHOW GRANT ROLE xxx ON [SERVER, DATABASE, TABLE, URI] xxx Diffs (updated) ----- sentry-binding/sentry-binding-hive/src/main/java/org/apache/hadoop/hive/ql/exec/SentryGrantRevokeTask.java 1012605e4d09074851704ecf9df4e4a613cd919d sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/SentryHiveAuthorizationTaskFactoryImpl.java 6d890411e5968bd6dd7e3f489bdf08d9842547d9 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java 50aadb868276f88ea6c36e23d83d118ca7205c44 Diff: https://reviews.apache.org/r/22105/diff/ Testing ------- Added new integration tests Thanks, Sravya Tirukkovalur