-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/51445/
-----------------------------------------------------------
Review request for sentry, Dapeng Sun, Hao Hao, and Sravya Tirukkovalur.
Bugs: SENTRY-1453
https://issues.apache.org/jira/browse/SENTRY-1453
Repository: sentry
Description
-------
As it currently stands user will have to set some sentry related configs in
Kafka's config and some in Sentry's configs. Configuration can be eased by
allowing users to pass all related configs in one place. One way is to enable
passing sentry client cache configs from Kafka conf.
Diffs
-----
sentry-binding/sentry-binding-kafka/src/main/java/org/apache/sentry/kafka/binding/KafkaAuthBinding.java
53e7a3c641698ee86cee1fa6f48e1ac4d396f412
sentry-binding/sentry-binding-kafka/src/main/java/org/apache/sentry/kafka/conf/KafkaAuthConf.java
3b1cb9cca017da62578cafef2ce273703482ffdd
sentry-tests/sentry-tests-kafka/src/main/java/org/apache/sentry/tests/e2e/kafka/KafkaTestServer.java
b82c0281db431067eb827ba559e6a231e255d780
sentry-tests/sentry-tests-kafka/src/test/java/org/apache/sentry/tests/e2e/kafka/AbstractKafkaSentryTestBase.java
10d6af73ddcab8c886377c80f0b3c604505c101a
Diff: https://reviews.apache.org/r/51445/diff/
Testing
-------
Tested with e2e kafka tests.
Thanks,
Ashish Singh
