[jira] [Comment Edited] (HIVE-6547) normalize struct Role in metastore thrift interface

["Thejas M Nair (JIRA)" <ji...@apache.org>]

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

Thejas M Nair edited comment on HIVE-6547 at 3/14/14 3:23 PM:
--------------------------------------------------------------

Thanks for the feedback. Will make them required.
While I am at it , i will also make the fields in GetPrincipalsInRoleResponse,GetPrincipalsInRoleRequest required. With the semantics of the functions, it does not make any sense for them to be non-required.
Those are similar functions that were added recently


was (Author: thejas):
Thanks for the feedback. Will make them required.
While I am at it , i will also make the fields in GetPrincipalsInRoleResponse,GetPrincipalsInRoleRequest required. With the semantics of the functions, it does not make any sense for them to be non-required.


> normalize struct Role in metastore thrift interface
> ---------------------------------------------------
>
>                 Key: HIVE-6547
>                 URL: https://issues.apache.org/jira/browse/HIVE-6547
>             Project: Hive
>          Issue Type: Bug
>          Components: Metastore, Thrift API
>    Affects Versions: 0.13.0
>            Reporter: Thejas M Nair
>            Assignee: Thejas M Nair
>             Fix For: 0.13.0
>
>         Attachments: HIVE-6547.thriftapi.patch
>
>
> As discussed in HIVE-5931, it will be cleaner to have the information about Role to role member mapping removed from the Role object, as it is not part of a logical Role. This information not relevant for actions such as creating a Role.
> As part of this change  get_role_grants_for_principal api will be added, so that it can be used in place of  list_roles, when role mapping information is desired.



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