----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31326/#review73675 -----------------------------------------------------------
agents-audit/src/main/java/org/apache/ranger/audit/provider/kafka/KafkaAuditProvider.java <https://reviews.apache.org/r/31326/#comment120065> Please add license text. agents-audit/src/main/java/org/apache/ranger/audit/provider/kafka/KafkaAuditProvider.java <https://reviews.apache.org/r/31326/#comment120066> It will be good to read the topic name from configuration. - Madhan Neethiraj On Feb. 23, 2015, 9:33 p.m., Don Bosco Durai wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31326/ > ----------------------------------------------------------- > > (Updated Feb. 23, 2015, 9:33 p.m.) > > > Review request for ranger. > > > Repository: ranger > > > Description > ------- > > Implement RangerAudit provider interface with Kafka Publisher > > > Diffs > ----- > > agents-audit/pom.xml b437bc1 > > agents-audit/src/main/java/org/apache/ranger/audit/provider/AuditProviderFactory.java > fb5e8b5 > > agents-audit/src/main/java/org/apache/ranger/audit/provider/BaseAuditProvider.java > 9a0cc45 > > agents-audit/src/main/java/org/apache/ranger/audit/provider/kafka/KafkaAuditProvider.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/31326/diff/ > > > Testing > ------- > > > Thanks, > > Don Bosco Durai > >
