> On Aug. 13, 2014, 7:44 p.m., Neha Narkhede wrote: > > core/src/main/scala/kafka/network/SocketServer.scala, line 470 > > <https://reviews.apache.org/r/24653/diff/1/?file=659368#file659368line470> > > > > While this may fix the NoSuchElementException, I wonder why the error > > happened in the first place as it may point to a bug in the way we are > > using the connection quotas. For example, are we trying to shut down the > > same socket channel twice somehow? > > > > Finding this will also help write a unit test for this, if possible.
Whitespaces issue fixed. As per comments by Joel Koshy this case is hard to reproduce I tried to write a test where sending bunch requests to socketserver and calling shutdown didn't reproduce this issue. - Sriharsha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24653/#review50495 ----------------------------------------------------------- On Aug. 21, 2014, 2:58 a.m., Sriharsha Chintalapani wrote: > > ----------------------------------------------------------- > 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 > https://issues.apache.org/jira/browse/KAFKA-1577 > > > Repository: kafka > > > Description > ------- > > KAFKA-1577. Exception in ConnectionQuotas while shutting down. > > > Diffs > ----- > > core/src/main/scala/kafka/network/SocketServer.scala > 9693bc0b2d92377ee378e68f25b522bcaa836ddf > > Diff: https://reviews.apache.org/r/24653/diff/ > > > Testing > ------- > > > Thanks, > > Sriharsha Chintalapani > >
