-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30575/
-----------------------------------------------------------
(Updated Feb. 4, 2015, 1:30 a.m.)
Review request for hive and Jason Dere.
Changes
-------
HIVE-9350.4.patch - addressing review comments from Jason
Bugs: HIVE-9350
https://issues.apache.org/jira/browse/HIVE-9350
Repository: hive-git
Description
-------
https://issues.apache.org/jira/browse/HIVE-9350
Diffs (updated)
-----
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 6d2748e
itests/hive-unit/src/test/java/org/apache/hadoop/hive/metastore/TestFilterHooks.java
cceac93
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/security/authorization/plugin/TestHiveAuthorizerShowFilters.java
PRE-CREATION
metastore/src/java/org/apache/hadoop/hive/metastore/DefaultMetaStoreFilterHookImpl.java
b723484
metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreFilterHook.java
51f63ad
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/AuthorizationMetaStoreFilterHook.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAccessControlException.java
d877686
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizationValidator.java
5a5b3d5
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizer.java
1f1eba2
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveAuthorizerImpl.java
e615049
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/HiveV1Authorizer.java
ac1cc47
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/DummyHiveAuthorizationValidator.java
cabc22a
ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/sqlstd/SQLStdHiveAuthorizationValidator.java
0e093b0
ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java d81b44c
service/src/java/org/apache/hive/service/cli/CLIService.java 883bf9b
Diff: https://reviews.apache.org/r/30575/diff/
Testing
-------
New unit tests.
Thanks,
Thejas Nair
