Re: Review Request 24653: Patch for KAFKA-1577

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24653/#review51905 --- Ship it! Ship It! - Joel Koshy On Aug. 26, 2014, 2:33 p.m., Srih

Re: Review Request 24653: Patch for KAFKA-1577

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24653/#review51891 --- core/src/main/scala/kafka/network/SocketServer.scala

Re: Review Request 24653: Patch for KAFKA-1577

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24653/ --- (Updated Aug. 26, 2014, 2:33 p.m.) Review request for kafka. Bugs: KAFKA-1577

Re: Review Request 24653: Patch for KAFKA-1577

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24653/#review51194 --- core/src/main/scala/kafka/network/SocketServer.scala

Re: Review Request 24653: Patch for KAFKA-1577

> On Aug. 13, 2014, 7:44 p.m., Neha Narkhede wrote: > > core/src/main/scala/kafka/network/SocketServer.scala, line 470 > > > > > > While this may fix the NoSuchElementException, I wonder why the error > > happened in

Re: Review Request 24653: Patch for KAFKA-1577

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24653/ --- (Updated Aug. 21, 2014, 2:58 a.m.) Review request for kafka. Bugs: KAFKA-1577

Re: Review Request 24653: Patch for KAFKA-1577

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24653/#review50495 --- I saw a bunch of whitespaces diffs. Do you know why those were intro

Review Request 24653: Patch for KAFKA-1577

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24653/ --- Review request for kafka. Bugs: KAFKA-1577 https://issues.apache.org/jira/b