----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26131/ -----------------------------------------------------------
Review request for sentry, Xiaomeng Huang, Arun Suresh, Colin Ma, Sun Dapeng, Prasad Mujumdar, and Sravya Tirukkovalur. 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
