[ 
https://issues.apache.org/jira/browse/HIVE-5931?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13920423#comment-13920423
 ] 

Ashutosh Chauhan commented on HIVE-5931:
----------------------------------------

Few comments on proposed api:
* Better name for 1st method : get_principals_in_role() ?
* Better name for 2nd method : get_roles_granted_to_principal() ?
* Also struct needs better name. Also, put explanation for struct, since it 
carries redundant info, depending on method it is used in. 
* principalType in struct should be enum

> SQL std auth - add metastore get_role_participants api - to support DESCRIBE 
> ROLE
> ---------------------------------------------------------------------------------
>
>                 Key: HIVE-5931
>                 URL: https://issues.apache.org/jira/browse/HIVE-5931
>             Project: Hive
>          Issue Type: Sub-task
>          Components: Authorization
>            Reporter: Thejas M Nair
>         Attachments: HIVE-5931.thriftapi.followup.patch, 
> HIVE-5931.thriftapi.patch
>
>   Original Estimate: 24h
>  Remaining Estimate: 24h
>
> This is necessary for DESCRIBE ROLE role statement. This will list
> all users and roles that participate in a role. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to