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

(Updated Oct. 22, 2014, 9:51 p.m.)


Review request for hive, Gunther Hagleitner and Jason Dere.


Changes
-------

 HIVE-8534.5.patch - addressing review comments


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


Repository: hive-git


Description
-------

https://issues.apache.org/jira/browse/HIVE-8534


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 6a9da7d 
  
itests/util/src/main/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessControllerForTest.java
 88004c8 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAccessController.java
 a18eedc 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizer.java
 9c3a95c 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizerImpl.java
 69b803a 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java
 PRE-CREATION 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessController.java
 658ff76 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAccessControllerWrapper.java
 29c1034 
  
ql/src/test/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/TestSQLStdHiveAccessControllerCLI.java
 5c4d713 
  
ql/src/test/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/TestSQLStdHiveAccessControllerHS2.java
 f13cf7e 
  ql/src/test/queries/clientnegative/authorization_disallow_transform.q 342c29a 
  ql/src/test/queries/clientnegative/authorization_set_invalidconf.q 
PRE-CREATION 
  ql/src/test/results/clientnegative/authorization_disallow_transform.q.out 
39819b6 
  ql/src/test/results/clientnegative/authorization_set_invalidconf.q.out 
PRE-CREATION 

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


Testing
-------

Tests updated


Thanks,

Thejas Nair

Reply via email to