You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Thejas Nair <th...@hortonworks.com> on 2014/08/12 19:42:46 UTC
Re: Review Request 24610: HIVE-7648 : authorization check api should provide
table for create table, drop/create index, and db for create/switch db
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24610/
-----------------------------------------------------------
(Updated Aug. 12, 2014, 5:42 p.m.)
Review request for hive and Jason Dere.
Summary (updated)
-----------------
HIVE-7648 : authorization check api should provide table for create table,drop/create index, and db for create/switch db
Bugs: HIVE-7648
https://issues.apache.org/jira/browse/HIVE-7648
Repository: hive-git
Description
-------
See https://issues.apache.org/jira/browse/HIVE-7648
Uploaded the 3.patch, excluding the .q.out file changes (except for new test cases).
Diffs
-----
hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestSemanticAnalysis.java a465562
ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 0e705aa
ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 3fcd54b
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/Operation2Privilege.java 4a3d8a7
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizationValidator.java 83a0b45
ql/src/test/queries/clientnegative/authorization_create_index.q PRE-CREATION
ql/src/test/queries/clientnegative/authorization_drop_index.q PRE-CREATION
ql/src/test/results/clientnegative/authorization_create_index.q.out PRE-CREATION
ql/src/test/results/clientnegative/authorization_drop_index.q.out PRE-CREATION
ql/src/test/results/clientpositive/authorization_explain.q.out 3aeb170
Diff: https://reviews.apache.org/r/24610/diff/
Testing
-------
New tests added, many q.out files updated.
Thanks,
Thejas Nair