hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Thejas Nair" <the...@hortonworks.com>
Subject Re: Review Request 17163: HIVE-5929 - SQL std auth - Access control statement updates
Date Tue, 28 Jan 2014 00:35:12 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17163/
-----------------------------------------------------------

(Updated Jan. 28, 2014, 12:35 a.m.)


Review request for hive and Ashutosh Chauhan.


Changes
-------

HIVE-5929.6.patch - address review comments


Bugs: HIVE-5929
    https://issues.apache.org/jira/browse/HIVE-5929


Repository: hive-git


Description
-------

Subtask for sql standard based auth, for performing the updates to metastore from newly supported
access  control statements .


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/Driver.java cb0c1a5 
  ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 92ed55b 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveUtils.java c65bf28 
  ql/src/java/org/apache/hadoop/hive/ql/parse/authorization/HiveAuthorizationTaskFactoryImpl.java
c41cd0f 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/AuthorizationUtils.java PRE-CREATION

  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/DefaultHiveAuthorizerFactory.java
7470e9d 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAccessController.java
8e4114f 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizationPluginException.java
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizationValidator.java
63046f5 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizer.java
c10a2ac 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizerImpl.java
ca95bfc 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrivilegeInfo.java
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizationValidator.java
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizerFactory.java
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 08f7fec 
  ql/src/test/queries/clientpositive/authorization_1_sql_std.q PRE-CREATION 
  ql/src/test/queries/clientpositive/authorization_role_grant1.q 1a375a5 
  ql/src/test/results/clientpositive/authorization_1_sql_std.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/authorization_role_grant1.q.out 496687c 

Diff: https://reviews.apache.org/r/17163/diff/


Testing
-------

.q tests included.


Thanks,

Thejas Nair


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