hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Thejas Nair" <the...@hortonworks.com>
Subject Re: Review Request 30575: HIVE-9350 : Add ability for HiveAuthorizer implementations to filter out results of 'show tables', 'show databases'
Date Tue, 03 Feb 2015 19:27:16 GMT

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

(Updated Feb. 3, 2015, 7:27 p.m.)


Review request for hive and Jason Dere.


Changes
-------

HIVE-9350.3.patch - remove unnecessary import that was added in patch from SessionManager


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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message