----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65777/#review198453 -----------------------------------------------------------
Ship it! Ship It! - Colm O hEigeartaigh On Feb. 27, 2018, 11:17 a.m., Zsombor Gegesy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65777/ > ----------------------------------------------------------- > > (Updated Feb. 27, 2018, 11:17 a.m.) > > > Review request for ranger. > > > 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 > ----- > > 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 294c422f > ranger-kafka-plugin-shim/pom.xml f1aeee6f > > > Diff: https://reviews.apache.org/r/65777/diff/3/ > > > Testing > ------- > > Unit & local tests > > > Thanks, > > Zsombor Gegesy > >
