You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Navis Ryu <na...@nexr.com> on 2014/05/08 04:29:03 UTC
Review Request 21191: Support newly added role related APIs for v1 authorizer
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21191/
-----------------------------------------------------------
Review request for hive.
Bugs: HIVE-7026
https://issues.apache.org/jira/browse/HIVE-7026
Repository: hive-git
Description
-------
Support SHOW_CURRENT_ROLE and SHOW_ROLE_PRINCIPALS for v1 authorizer.
Diffs
-----
metastore/src/java/org/apache/hadoop/hive/metastore/Warehouse.java 6d9b559
ql/src/java/org/apache/hadoop/hive/ql/ErrorMsg.java bc18ef5
ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java ca51e71
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/AuthorizationUtils.java a95d784
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/Privilege.java f9d1b4b
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrincipal.java 62b8994
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrivilege.java 126300a
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrivilegeObject.java a3a689d
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveV1Authorizer.java PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java e4f5aac
ql/src/test/results/clientpositive/authorization_1.q.out 0ff4b04
ql/src/test/results/clientpositive/authorization_2.q.out 9acb42b
ql/src/test/results/clientpositive/authorization_3.q.out 834909c
ql/src/test/results/clientpositive/authorization_4.q.out 1b745d8
ql/src/test/results/clientpositive/authorization_5.q.out 6f75dd1
ql/src/test/results/clientpositive/authorization_6.q.out 41696b1
ql/src/test/results/clientpositive/authorization_9.q.out 226ce28
Diff: https://reviews.apache.org/r/21191/diff/
Testing
-------
Thanks,
Navis Ryu
Re: Review Request 21191: Support newly added role related APIs for v1
authorizer
Posted by Navis Ryu <na...@nexr.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21191/
-----------------------------------------------------------
(Updated July 21, 2014, 6:41 a.m.)
Review request for hive and Thejas Nair.
Bugs: HIVE-7026
https://issues.apache.org/jira/browse/HIVE-7026
Repository: hive-git
Description
-------
Support SHOW_CURRENT_ROLE and SHOW_ROLE_PRINCIPALS for v1 authorizer.
Diffs
-----
metastore/src/java/org/apache/hadoop/hive/metastore/Warehouse.java 82b871b
ql/src/java/org/apache/hadoop/hive/ql/ErrorMsg.java 9889cfe
ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 40ec4e5
ql/src/java/org/apache/hadoop/hive/ql/metadata/SessionHiveMetaStoreClient.java 37b1669
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/AuthorizationUtils.java 604c39d
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/Privilege.java f9d1b4b
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrincipal.java dd4cd22
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrivilege.java 126300a
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrivilegeObject.java 8cdff5b
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveV1Authorizer.java PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLAuthorizationUtils.java 6b635ce
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java 932b980
ql/src/test/queries/clientnegative/authorization_show_role_principals_v1.q 69cea2f
ql/src/test/queries/clientpositive/authorization_show_role_principals_v1.q PRE-CREATION
ql/src/test/results/clientnegative/authorization_caseinsensitivity.q.out 9cd36b6
ql/src/test/results/clientnegative/authorization_fail_1.q.out 7af4ae8
ql/src/test/results/clientnegative/authorization_fail_3.q.out d3b2a7c
ql/src/test/results/clientnegative/authorization_fail_4.q.out 3ce149b
ql/src/test/results/clientnegative/authorization_fail_5.q.out 72b074f
ql/src/test/results/clientnegative/authorization_fail_7.q.out e9b7c35
ql/src/test/results/clientnegative/authorization_fail_8.q.out 10dd71b
ql/src/test/results/clientnegative/authorization_part.q.out dfdcb39
ql/src/test/results/clientnegative/authorization_public_create.q.out 0318a8b
ql/src/test/results/clientnegative/authorization_public_drop.q.out 912589d
ql/src/test/results/clientnegative/authorization_role_cycles1.q.out 704b7b1
ql/src/test/results/clientnegative/authorization_role_cycles2.q.out 03348a8
ql/src/test/results/clientnegative/authorization_role_grant.q.out a497467
ql/src/test/results/clientnegative/authorization_role_grant2.q.out dc662ec
ql/src/test/results/clientnegative/authorization_role_grant_nosuchrole.q.out 6193103
ql/src/test/results/clientnegative/authorization_show_role_principals_v1.q.out e9a965b
ql/src/test/results/clientpositive/alter_rename_partition_authorization.q.out 1e00d1b
ql/src/test/results/clientpositive/authorization_1.q.out 0ff4b04
ql/src/test/results/clientpositive/authorization_2.q.out 5d82e56
ql/src/test/results/clientpositive/authorization_3.q.out 834909c
ql/src/test/results/clientpositive/authorization_4.q.out 1b745d8
ql/src/test/results/clientpositive/authorization_5.q.out 6f75dd1
ql/src/test/results/clientpositive/authorization_6.q.out b7bccd0
ql/src/test/results/clientpositive/authorization_9.q.out d03682c
ql/src/test/results/clientpositive/authorization_show_role_principals_v1.q.out PRE-CREATION
ql/src/test/results/clientpositive/keyword_1.q.out 84bb2d0
ql/src/test/results/clientpositive/show_roles.q.out f90bba0
Diff: https://reviews.apache.org/r/21191/diff/
Testing
-------
Thanks,
Navis Ryu
Re: Review Request 21191: Support newly added role related APIs for v1
authorizer
Posted by Navis Ryu <na...@nexr.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21191/
-----------------------------------------------------------
(Updated July 21, 2014, 6:41 a.m.)
Review request for hive.
Changes
-------
Rebased to trunk
Bugs: HIVE-7026
https://issues.apache.org/jira/browse/HIVE-7026
Repository: hive-git
Description
-------
Support SHOW_CURRENT_ROLE and SHOW_ROLE_PRINCIPALS for v1 authorizer.
Diffs (updated)
-----
metastore/src/java/org/apache/hadoop/hive/metastore/Warehouse.java 82b871b
ql/src/java/org/apache/hadoop/hive/ql/ErrorMsg.java 9889cfe
ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 40ec4e5
ql/src/java/org/apache/hadoop/hive/ql/metadata/SessionHiveMetaStoreClient.java 37b1669
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/AuthorizationUtils.java 604c39d
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/Privilege.java f9d1b4b
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrincipal.java dd4cd22
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrivilege.java 126300a
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrivilegeObject.java 8cdff5b
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveV1Authorizer.java PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLAuthorizationUtils.java 6b635ce
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java 932b980
ql/src/test/queries/clientnegative/authorization_show_role_principals_v1.q 69cea2f
ql/src/test/queries/clientpositive/authorization_show_role_principals_v1.q PRE-CREATION
ql/src/test/results/clientnegative/authorization_caseinsensitivity.q.out 9cd36b6
ql/src/test/results/clientnegative/authorization_fail_1.q.out 7af4ae8
ql/src/test/results/clientnegative/authorization_fail_3.q.out d3b2a7c
ql/src/test/results/clientnegative/authorization_fail_4.q.out 3ce149b
ql/src/test/results/clientnegative/authorization_fail_5.q.out 72b074f
ql/src/test/results/clientnegative/authorization_fail_7.q.out e9b7c35
ql/src/test/results/clientnegative/authorization_fail_8.q.out 10dd71b
ql/src/test/results/clientnegative/authorization_part.q.out dfdcb39
ql/src/test/results/clientnegative/authorization_public_create.q.out 0318a8b
ql/src/test/results/clientnegative/authorization_public_drop.q.out 912589d
ql/src/test/results/clientnegative/authorization_role_cycles1.q.out 704b7b1
ql/src/test/results/clientnegative/authorization_role_cycles2.q.out 03348a8
ql/src/test/results/clientnegative/authorization_role_grant.q.out a497467
ql/src/test/results/clientnegative/authorization_role_grant2.q.out dc662ec
ql/src/test/results/clientnegative/authorization_role_grant_nosuchrole.q.out 6193103
ql/src/test/results/clientnegative/authorization_show_role_principals_v1.q.out e9a965b
ql/src/test/results/clientpositive/alter_rename_partition_authorization.q.out 1e00d1b
ql/src/test/results/clientpositive/authorization_1.q.out 0ff4b04
ql/src/test/results/clientpositive/authorization_2.q.out 5d82e56
ql/src/test/results/clientpositive/authorization_3.q.out 834909c
ql/src/test/results/clientpositive/authorization_4.q.out 1b745d8
ql/src/test/results/clientpositive/authorization_5.q.out 6f75dd1
ql/src/test/results/clientpositive/authorization_6.q.out b7bccd0
ql/src/test/results/clientpositive/authorization_9.q.out d03682c
ql/src/test/results/clientpositive/authorization_show_role_principals_v1.q.out PRE-CREATION
ql/src/test/results/clientpositive/keyword_1.q.out 84bb2d0
ql/src/test/results/clientpositive/show_roles.q.out f90bba0
Diff: https://reviews.apache.org/r/21191/diff/
Testing
-------
Thanks,
Navis Ryu