----------------------------------------------------------- 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