Re: Review Request 60816: Addresses Use of KafkaConsumer in Thread by Implementing Kafka's ShutdownableThread

2017-07-13 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60816/#review180449 --- Ship it! Ship It! - Sarath Subramanian On July 12, 2017, 10:

Re: Review Request 60816: Addresses Use of KafkaConsumer in Thread by Implementing Kafka's ShutdownableThread

2017-07-12 Thread Ayub Pathan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60816/#review180406 --- Ship it! Looks good to me! - Ayub Pathan On July 13, 2017, 5

Re: Review Request 60816: Addresses Use of KafkaConsumer in Thread by Implementing Kafka's ShutdownableThread

2017-07-12 Thread Nixon Rodrigues
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60816/#review180404 --- Ship it! Changes looks good to me. I have validated this change

Re: Review Request 60816: Addresses Use of KafkaConsumer in Thread by Implementing Kafka's ShutdownableThread

2017-07-12 Thread Apoorv Naik
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60816/#review180401 --- Ship it! Looks good to me. Will commit after another contributo

Re: Review Request 60816: Addresses Use of KafkaConsumer in Thread by Implementing Kafka's ShutdownableThread

2017-07-12 Thread Ashutosh Mestry
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60816/ --- (Updated July 13, 2017, 5:07 a.m.) Review request for atlas, Ayub Pathan, Madha

Review Request 60816: Addresses Use of KafkaConsumer in Thread by Implementing Kafka's ShutdownableThread

2017-07-12 Thread Ashutosh Mestry
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60816/ --- Review request for atlas, Ayub Pathan, Madhan Neethiraj, and Nixon Rodrigues. B