Re: Review Request 33731: KAFKA-2160

2015-04-30 Thread Onur Karaman
> On April 30, 2015, 10:45 p.m., Onur Karaman wrote: > > core/src/main/scala/kafka/server/DelayedOperation.scala, line 224 > > > > > > We can put the key inside Watchers and just use the watchersForKey > > that's alre

Re: Review Request 33731: KAFKA-2160

2015-04-30 Thread Onur Karaman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33731/#review82217 --- core/src/main/scala/kafka/server/DelayedOperation.scala

Re: Review Request 33731: KAFKA-2160

2015-04-30 Thread Guozhang Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33731/ --- (Updated April 30, 2015, 10:20 p.m.) Review request for kafka. Bugs: KAFKA-21

Re: Review Request 33731: KAFKA-2160

2015-04-30 Thread Onur Karaman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33731/#review82204 --- core/src/main/scala/kafka/server/DelayedOperation.scala

Review Request 33731: KAFKA-2160

2015-04-30 Thread Guozhang Wang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33731/ --- Review request for kafka. Bugs: KAFKA-2160 https://issues.apache.org/jira/b