ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Zsombor Gegesy <gzsom...@gmail.com>
Subject Re: Review Request 65777: RANGER-1967 - Kafka 1.0 support
Date Sat, 24 Feb 2018 10:08:30 GMT

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

(Updated Feb. 24, 2018, 10:08 a.m.)


Review request for ranger.


Changes
-------

Apache header for log4j.properties


Bugs: RANGER-1967
    https://issues.apache.org/jira/browse/RANGER-1967


Repository: ranger


Description
-------

The API changes are not too big, however running Kafka in embedded mode turned out to be a
bit trickier beast. I had to add a couple of new flags, otherwise it would wait for 2 other
broker to join.
 The commit also contains logging changes, even a test log4j configuration, just to make the
debugging easier.


Diffs (updated)
-----

  agents-audit/pom.xml 4fa44c5f 
  plugin-kafka/pom.xml f644646b 
  plugin-kafka/src/main/java/org/apache/ranger/authorization/kafka/authorizer/RangerKafkaAuthorizer.java
b3d5a74d 
  plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerGSSTest.java
4ea39ed7 
  plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerSASLSSLTest.java
fb541cd3 
  plugin-kafka/src/test/java/org/apache/ranger/authorization/kafka/authorizer/KafkaRangerAuthorizerTest.java
fb0a2c0f 
  plugin-kafka/src/test/resources/log4j.properties PRE-CREATION 
  pom.xml fa1d1c7b 
  ranger-kafka-plugin-shim/pom.xml f1aeee6f 


Diff: https://reviews.apache.org/r/65777/diff/2/

Changes: https://reviews.apache.org/r/65777/diff/1-2/


Testing
-------

Unit & local tests


Thanks,

Zsombor Gegesy


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message