Re: Review Request 26564: Patch for KAFKA-1471

2014-10-12 Thread Neha Narkhede
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26564/#review56365 --- Thanks for updating the patch, Ewen. 2 minor comments. Probably

Re: Review Request 26564: Patch for KAFKA-1471

2014-10-12 Thread Ewen Cheslack-Postava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26564/ --- (Updated Oct. 12, 2014, 11:02 p.m.) Review request for kafka. Bugs:

Re: Review Request 26564: Patch for KAFKA-1471

2014-10-12 Thread Neha Narkhede
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26564/#review56368 --- Ship it! Ship It! - Neha Narkhede On Oct. 12, 2014, 11:02 p.m.,

Re: Review Request 26564: Patch for KAFKA-1471

2014-10-12 Thread Neha Narkhede
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26564/#review56369 --- Ship it! Ship It! - Neha Narkhede On Oct. 12, 2014, 11:02 p.m.,

Review Request 26564: Patch for KAFKA-1471

2014-10-10 Thread Ewen Cheslack-Postava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26564/ --- Review request for kafka. Bugs: KAFKA-1471