-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19980/
-----------------------------------------------------------
Review request for sentry, Brock Noland, Prasad Mujumdar, Shreepadma
Venugopalan, and Xuefu Zhang.
Bugs: sentry-165
https://issues.apache.org/jira/browse/sentry-165
Repository: sentry
Description
-------
HIVE-6793 added a new function, createShowRolesTask() to the
HiveAuthorizationTaskFactory interface. I have basically mimicked the upstream
implementation here:
https://issues.apache.org/jira/secure/attachment/12637896/HIVE-6793.patch
But the only thing I am not sure about is this part:
new RoleDDLDesc(null, null, RoleDDLDesc.RoleOperation.SHOW_ROLES, null);
principalType, principalName, roleName = null.
Looks like we should be getting these values from the ast as in here?
https://github.com/apache/incubator-sentry/blob/master/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/SentryHiveAuthorizationTaskFactoryImpl.java#L85
Diffs
-----
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/SentryHiveAuthorizationTaskFactoryImpl.java
de0e460a982dc72f7f2dff3f53ea80cbc6945475
Diff: https://reviews.apache.org/r/19980/diff/
Testing
-------
Compile and existing tests pass.
Thanks,
Sravya Tirukkovalur
