[ 
https://issues.apache.org/jira/browse/SENTRY-912?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15108044#comment-15108044
 ] 

Dapeng Sun commented on SENTRY-912:
-----------------------------------

Hi [~singhashish], thank you for your updated, I have a quick look at your 
patch,

1. I think it is better to split the jira to several subtasks, core, policy, 
binding, E2E and etc. It is better for other committers to review such a big 
task.
2. A patch with version should be uploaded to jira, for example 
SENTRY-912.001.patch, Jenkins will test it.
3. RB should add group {{sentry}}, so that sentry community can receive the 
update of review. 
4. I will comment at RB for some small issues.

> Sentry integration with Apache Kafka
> ------------------------------------
>
>                 Key: SENTRY-912
>                 URL: https://issues.apache.org/jira/browse/SENTRY-912
>             Project: Sentry
>          Issue Type: New Feature
>            Reporter: Dapeng Sun
>            Assignee: Ashish K Singh
>             Fix For: 1.7.0
>
>         Attachments: Design-SENTRY-912 Kafka Intergration-20151014.pdf
>
>
> Apache Kafka supports pluggable authorization by configuring a Kafka 
> Authorizer now(https://issues.apache.org/jira/browse/KAFKA-2210), This jira 
> will design and implementation a Sentry Kafka Authorizer, so that Kafka could 
> benefit from Sentry policy engine.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to