-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26373/
-----------------------------------------------------------
(Updated Oct. 28, 2014, 12:20 a.m.)
Review request for kafka.
Bugs: KAFKA-1647
https://issues.apache.org/jira/browse/KAFKA-1647
Repository: kafka
Description
-------
Addressed Joel's comments.
the version 2 code seems to be submitted by mistake... This should be the code
for review that addressed Joel's comments.
Addressed Jun's comments. Will do tests to verify if it works.
Addressed Joel's comments, we do not need to check the if leader exits for not
when adding fetcher.
Diffs
-----
core/src/main/scala/kafka/server/ReplicaManager.scala
78b7514cc109547c562e635824684fad581af653
Diff: https://reviews.apache.org/r/26373/diff/
Testing (updated)
-------
Followed Joel's testing step. I was able to reproduce the problem without the
patch and the WARN message goes away after applied the patch.
Thanks,
Jiangjie Qin
