----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59799/#review177095 -----------------------------------------------------------
distro/src/conf/atlas-application.properties Lines 81 (patched) <https://reviews.apache.org/r/59799/#comment250643> why is auto commit set to true. This will cause issues when there is a transient failure on Atlas server and no retries will be attempted - Suma Shivaprasad On June 5, 2017, 8:30 a.m., Nixon Rodrigues wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59799/ > ----------------------------------------------------------- > > (Updated June 5, 2017, 8:30 a.m.) > > > Review request for atlas, Apoorv Naik, Ashutosh Mestry, Madhan Neethiraj, and > Sarath Subramanian. > > > Repository: atlas > > > Description > ------- > > This patch include changes that implements kafka new-consumer api for > kafkaConsumer for Atlas Notification System. > > > Diffs > ----- > > distro/src/conf/atlas-application.properties 5e59528 > notification/src/main/java/org/apache/atlas/kafka/AtlasKafkaConsumer.java > PRE-CREATION > notification/src/main/java/org/apache/atlas/kafka/KafkaConsumer.java > 16c0eb2 > notification/src/main/java/org/apache/atlas/kafka/KafkaNotification.java > 8bd31fd > > notification/src/main/java/org/apache/atlas/notification/AbstractMessageDeserializer.java > 9585827 > > notification/src/main/java/org/apache/atlas/notification/AbstractNotificationConsumer.java > d4d78de > > notification/src/main/java/org/apache/atlas/notification/NotificationConsumer.java > a99cb10 > > webapp/src/main/java/org/apache/atlas/notification/NotificationHookConsumer.java > 2f8245d > > > Diff: https://reviews.apache.org/r/59799/diff/1/ > > > Testing > ------- > > Tested Kafka consumer on local embeded kafka /zookeeper env. > Tested Atlas Hive hook notifications. > Excecute Quick Start script. > > Due to change in kafkaconsumer api the current notification related UT/IT's > need to changed, i have not included this change in this patch but will > included it shortly. > > > Thanks, > > Nixon Rodrigues > >