hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Navis Ryu" <navis....@nexr.com>
Subject Re: Review Request 21191: Support newly added role related APIs for v1 authorizer
Date Mon, 21 Jul 2014 06:41:45 GMT

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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message