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

(Updated 一月 13, 2015, 7:19 p.m.)


Review request for sentry, Xiaomeng Huang, Arun Suresh, Colin Ma, shen guoquan, 
Lenni Kuff, Prasad Mujumdar, and Sravya Tirukkovalur.


Changes
-------

Update patch


Bugs: SENTRY-590
    https://issues.apache.org/jira/browse/SENTRY-590


Repository: sentry


Description
-------

Base on SENTRY-600: To use clientFactory, **SentryGenericServiceClientFactory** 
will use for create generic model client.
**SentryGenericServiceClient** need to change to interface, it's a limitation 
of dynamic proxy.


Diffs (updated)
-----

  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceClient.java
 9f4a292 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceClientDefaultImpl.java
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceClientFactory.java
 PRE-CREATION 
  
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/thrift/TestSentryGenericServiceIntegration.java
 dfd9f10 

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


Testing
-------

Unit Test in local


Thanks,

Dapeng Sun

Reply via email to