> On Feb. 21, 2015, 6:14 p.m., Neha Narkhede wrote: > > core/src/main/scala/kafka/server/KafkaHealthcheck.scala, line 53 > > <https://reviews.apache.org/r/31260/diff/1/?file=871715#file871715line53> > > > > We should try to understand why we need to explicitly de-register here. > > It isn't immediately clear to me.
If you understand it, please can you add a comment here? - Neha ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31260/#review73411 ----------------------------------------------------------- On Feb. 21, 2015, 4:16 p.m., Jun Rao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31260/ > ----------------------------------------------------------- > > (Updated Feb. 21, 2015, 4:16 p.m.) > > > Review request for kafka. > > > Bugs: kafka-1971 > https://issues.apache.org/jira/browse/kafka-1971 > > > Repository: kafka > > > Description > ------- > > KAFKA-1971; starting a broker with a conflicting id will delete the previous > broker registration > > > Diffs > ----- > > core/src/main/scala/kafka/server/KafkaHealthcheck.scala > 4acdd70fe9c1ee78d6510741006c2ece65450671 > > Diff: https://reviews.apache.org/r/31260/diff/ > > > Testing > ------- > > > Thanks, > > Jun Rao > >
