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

(Updated Feb. 22, 2014, 12:54 a.m.)


Review request for hive and Thejas Nair.


Changes
-------

Only restrict create/drop of metastore functions. temp functions/macros not 
affected.


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


Repository: hive-git


Description
-------

Add output entity of DB object to make sure only admin roles can add/drop 
functions/macros.


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/parse/FunctionSemanticAnalyzer.java 
68a25e0 
  
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java
 7dfd574 
  ql/src/test/queries/clientnegative/authorization_create_func1.q PRE-CREATION 
  ql/src/test/queries/clientpositive/authorization_create_func1.q PRE-CREATION 
  ql/src/test/results/clientnegative/authorization_create_func1.q.out 
PRE-CREATION 
  ql/src/test/results/clientnegative/create_function_nonexistent_class.q.out 
393a3e8 
  ql/src/test/results/clientnegative/create_function_nonexistent_db.q.out 
ebb069e 
  ql/src/test/results/clientnegative/create_function_nonudf_class.q.out dd66afc 
  ql/src/test/results/clientnegative/udf_local_resource.q.out b6ea77d 
  ql/src/test/results/clientnegative/udf_nonexistent_resource.q.out ad70d54 
  ql/src/test/results/clientpositive/authorization_create_func1.q.out 
PRE-CREATION 
  ql/src/test/results/clientpositive/create_func1.q.out 5a249c3 
  ql/src/test/results/clientpositive/udf_using.q.out 69e5f3b 

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


Testing
-------

positive/negative q files added


Thanks,

Jason Dere

Reply via email to