-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26131/
-----------------------------------------------------------
(Updated 九月 29, 2014, 8:44 a.m.)
Review request for sentry, Xiaomeng Huang, Arun Suresh, Colin Ma, Sun Dapeng,
Prasad Mujumdar, and Sravya Tirukkovalur.
Summary (updated)
-----------------
SENTRY-404:Extending Sentry thrift interface and adding a processor for generic
authorization model
Bugs: sentry-404
https://issues.apache.org/jira/browse/sentry-404
Repository: sentry
Description
-------
The previous thrift interface is not general and mainly fit for database
authorization model,like hive/impala. If the sentry want to binding the other
no-database authorization model such as Solr/ElasticSearch, the current sentry
service can't handle this requirement. So there is a requirement to extend the
sentry thrift interface to adapt the generic authorization model and add a new
processor to handle thrift requests.
Diffs
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/genericModel/service/thrift/NotificationHandler.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/genericModel/service/thrift/NotificationHandlerInvoker.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/genericModel/service/thrift/SentryGenericPolicyProcessor.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/genericModel/service/thrift/SentryGenericPolicyProcessorFactory.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/genericModel/service/thrift/SentryGenericServiceClient.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/PolicyStoreConstants.java
34bec93
sentry-provider/sentry-provider-db/src/main/resources/sentry_common_service.thrift
9456274
sentry-provider/sentry-provider-db/src/main/resources/sentry_generic_policy_service.thrift
PRE-CREATION
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/genericModel/service/thrift/TestNotificationHandlerInvoker.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/genericModel/service/thrift/TestSentryGenericPolicyProcessor.java
PRE-CREATION
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/genericModel/service/thrift/TestSentryGenericServiceIntegration.java
PRE-CREATION
Diff: https://reviews.apache.org/r/26131/diff/
Testing
-------
Thanks,
shen guoquan
