You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by ru jia <82...@163.com> on 2020/10/10 09:53:24 UTC
Review Request 72947: RANGER-3033:Ranger hive authorizer should impl
'getRoleGrantInfoForPrincipal' interface
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72947/
-----------------------------------------------------------
Review request for ranger.
Bugs: RANGER-3033
https://issues.apache.org/jira/browse/RANGER-3033
Repository: ranger
Description
-------
Currently, hive authorizer doesn't support command 'show role grant user user1'
Diffs
-----
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java 1bec50b37
Diff: https://reviews.apache.org/r/72947/diff/1/
Testing
-------
1. create a role which contains user 'testuser' on Ranger web
2. go into beeline(make sure kerberos login in security mode)
3. execute 'show role grant user testuser', and then console will print the detail out
Thanks,
ru jia
Re: Review Request 72947: RANGER-3033:Ranger hive authorizer should
impl 'getRoleGrantInfoForPrincipal' interface
Posted by ru jia <82...@163.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72947/
-----------------------------------------------------------
(Updated 十月 13, 2020, 8:37 a.m.)
Review request for ranger.
Bugs: RANGER-3033
https://issues.apache.org/jira/browse/RANGER-3033
Repository: ranger
Description
-------
Currently, hive authorizer doesn't support command 'show role grant user user1'
Diffs (updated)
-----
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java 1bec50b37
Diff: https://reviews.apache.org/r/72947/diff/2/
Changes: https://reviews.apache.org/r/72947/diff/1-2/
Testing
-------
1. create a role which contains user 'testuser' on Ranger web
2. go into beeline(make sure kerberos login in security mode)
3. execute 'show role grant user testuser', and then console will print the detail out
Thanks,
ru jia
Re: Review Request 72947: RANGER-3033:Ranger hive authorizer should
impl 'getRoleGrantInfoForPrincipal' interface
Posted by bhavik patel <bh...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72947/#review222018
-----------------------------------------------------------
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java
Lines 2276 (patched)
<https://reviews.apache.org/r/72947/#comment311090>
Please enclose the debug statement inside if and update the logging format "className.MethodName".
hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java
Lines 2301 (patched)
<https://reviews.apache.org/r/72947/#comment311091>
Move this block at line 2309
- bhavik patel
On Oct. 10, 2020, 9:53 a.m., ru jia wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72947/
> -----------------------------------------------------------
>
> (Updated Oct. 10, 2020, 9:53 a.m.)
>
>
> Review request for ranger.
>
>
> Bugs: RANGER-3033
> https://issues.apache.org/jira/browse/RANGER-3033
>
>
> Repository: ranger
>
>
> Description
> -------
>
> Currently, hive authorizer doesn't support command 'show role grant user user1'
>
>
> Diffs
> -----
>
> hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java 1bec50b37
>
>
> Diff: https://reviews.apache.org/r/72947/diff/1/
>
>
> Testing
> -------
>
> 1. create a role which contains user 'testuser' on Ranger web
> 2. go into beeline(make sure kerberos login in security mode)
> 3. execute 'show role grant user testuser', and then console will print the detail out
>
>
> Thanks,
>
> ru jia
>
>