-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29418/
-----------------------------------------------------------
(Updated 一月 13, 2015, 7:17 p.m.)
Review request for sentry, Xiaomeng Huang, Arun Suresh, Colin Ma, shen guoquan,
Lenni Kuff, Prasad Mujumdar, and Sravya Tirukkovalur.
Changes
-------
Updated patch according Lenni's comments
Bugs: SENTRY-600
https://issues.apache.org/jira/browse/SENTRY-600
Repository: sentry
Description
-------
In order to increase the extensibility of ClientFactory, more client support
and more implementation of factory, use dynamic proxy to build the client
factory.
**SentryPolicyServiceBaseClient** and **SentryServiceClientFactory** is the
common Interface for client and factory.
**SentryServiceClientProxyFactory** will handle the integration of different
factorys.
Diffs (updated)
-----
sentry-binding/sentry-binding-hive/src/main/java/org/apache/hadoop/hive/ql/exec/SentryGrantRevokeTask.java
69e97a6
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/SentryConfigTool.java
ecbd664
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java
cc433d4
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/SimpleDBProviderBackend.java
ea8eb79
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/HAContext.java
523261e
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceBaseClient.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClient.java
962228f
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClientDefaultImpl.java
d21a401
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HAClientInvocationHandler.java
c6e265f
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryServiceClientFactory.java
574f23c
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryServicePolicyClientFactory.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/factory/SentryClientInvocationHandler.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/factory/SentryServiceClientFactory.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/factory/SentryServiceClientProxyFactory.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/factory/ha/HAClientInvocationHandler.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/factory/ha/SentryHAClientFactory.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java
0bdc3a2
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/AbstractTestWithDbProvider.java
04f50ed
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
f8cc1d0
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImport.java
7ebc0e4
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/metastore/AbstractMetastoreTestWithStaticConfiguration.java
9d15c95
Diff: https://reviews.apache.org/r/29418/diff/
Testing
-------
Unit Tests passed
Thanks,
Dapeng Sun
