> On April 25, 2015, 5:11 a.m., Guozhang Wang wrote: > > clients/src/test/java/org/apache/kafka/clients/MetadataTest.java, line 58 > > <https://reviews.apache.org/r/33125/diff/2/?file=926187#file926187line58> > > > > Do we need this sleep?
Guozhang, Sorry, had missed this note. The sleep avoids a tight loop in the main test thread while it is waiting for the other two threads to complete. - Rajini ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33125/#review81581 ----------------------------------------------------------- On April 13, 2015, 7:15 p.m., Rajini Sivaram wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33125/ > ----------------------------------------------------------- > > (Updated April 13, 2015, 7:15 p.m.) > > > Review request for kafka. > > > Bugs: KAFKA-2089 > https://issues.apache.org/jira/browse/KAFKA-2089 > > > Repository: kafka > > > Description > ------- > > Patch for KAFKA-2089: Fix timing issue in MetadataTest > > > Diffs > ----- > > clients/src/test/java/org/apache/kafka/clients/MetadataTest.java > 928087d29deb80655ca83726c1ebc45d76468c1f > > Diff: https://reviews.apache.org/r/33125/diff/ > > > Testing > ------- > > > Thanks, > > Rajini Sivaram > >
