-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17732/
-----------------------------------------------------------
Review request for hive.
Bugs: HIVE-6204
https://issues.apache.org/jira/browse/HIVE-6204
Repository: hive-git
Description
-------
{noformat}
hive> show grant role role1 on all;
OK
database default
table src
principalName role1
principalType ROLE
privilege Create
grantTime Wed Dec 18 14:17:56 KST 2013
grantor navis
database default
table srcpart
principalName role1
principalType ROLE
privilege Update
grantTime Wed Dec 18 14:18:28 KST 2013
grantor navis
{noformat}
This should be something like below, especially for JDBC clients.
{noformat}
hive> show grant role role1 on all;
OK
default src role1 ROLE Create false 1387343876000
navis
default srcpart role1 ROLE Update false 1387343908000
navis
{noformat}
Diffs
-----
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 89c9349
data/conf/hive-site.xml 88f3bda
metastore/if/hive_metastore.thrift e327e2a
metastore/src/gen/thrift/gen-cpp/hive_metastore_types.h a17bbf7
metastore/src/gen/thrift/gen-cpp/hive_metastore_types.cpp eedce50
metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/Role.java
35fcf58
metastore/src/gen/thrift/gen-php/metastore/Types.php dc914a5
metastore/src/gen/thrift/gen-py/hive_metastore/ttypes.py 6abfad4
metastore/src/gen/thrift/gen-rb/hive_metastore_types.rb 5e12ef7
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
2d8e483
ql/src/java/org/apache/hadoop/hive/ql/Context.java 9f7fd5d
ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java a926f1e
ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java f4d9a83
ql/src/java/org/apache/hadoop/hive/ql/plan/RoleDDLDesc.java de323a9
ql/src/java/org/apache/hadoop/hive/ql/plan/ShowGrantDesc.java de59ec3
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAccessController.java
9fe2a06
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizer.java
5a16a4a
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizerImpl.java
44dd161
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveRole.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java
adbd1a8
ql/src/test/results/clientnegative/authorization_fail_3.q.out 10df438
ql/src/test/results/clientnegative/authorization_fail_4.q.out 91ecbe9
ql/src/test/results/clientnegative/authorization_fail_5.q.out 7c70c66
ql/src/test/results/clientnegative/authorization_fail_7.q.out 7bfb6e6
ql/src/test/results/clientnegative/authorization_part.q.out 08866b4
ql/src/test/results/clientpositive/authorization_1.q.out 3391bcd
ql/src/test/results/clientpositive/authorization_1_sql_std.q.out a0b1ce0
ql/src/test/results/clientpositive/authorization_2.q.out 5b29855
ql/src/test/results/clientpositive/authorization_3.q.out ff42dfb
ql/src/test/results/clientpositive/authorization_4.q.out e656213
ql/src/test/results/clientpositive/authorization_5.q.out 3aeaeca
ql/src/test/results/clientpositive/authorization_6.q.out 0d555e4
ql/src/test/results/clientpositive/authorization_9.q.out bad418c
ql/src/test/results/clientpositive/authorization_role_grant1.q.out 981060f
ql/src/test/results/clientpositive/authorization_view.q.out 23a1c0b
Diff: https://reviews.apache.org/r/17732/diff/
Testing
-------
Thanks,
Navis Ryu
