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

(Updated May 22, 2014, 6:35 p.m.)


Review request for hive, Ashutosh Chauhan and Thejas Nair.


Changes
-------

Addressing review comments.
Fixed the one related failed test case (user name was not set, so the test 
output had my name in 'show grants' output).


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


Repository: hive-git


Description
-------

See bug


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/Driver.java 9040d9b 
  ql/src/java/org/apache/hadoop/hive/ql/hooks/WriteEntity.java 4276b3f 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/AuthorizationUtils.java
 a95d784 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HivePrivilegeObject.java
 a3a689d 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java
 b0a804c 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizationValidator.java
 229c063 
  ql/src/test/queries/clientnegative/authorization_insertoverwrite_nodel.q 
PRE-CREATION 
  ql/src/test/queries/clientpositive/authorization_insert.q PRE-CREATION 
  ql/src/test/results/clientnegative/authorization_insert_noinspriv.q.out 
ee8d49e 
  ql/src/test/results/clientnegative/authorization_insertoverwrite_nodel.q.out 
PRE-CREATION 
  ql/src/test/results/clientpositive/authorization_insert.q.out PRE-CREATION 

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


Testing
-------

Test included.


Thanks,

Thejas Nair

Reply via email to