[ 
https://issues.apache.org/jira/browse/KAFKA-575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13486079#comment-13486079
 ] 

Jun Rao commented on KAFKA-575:
-------------------------------

Patch looks good. Just one comment:

1. ControllerBrokerRequestBatch.sendRequestsToBrokers(): Instead of including 
all live brokers in the leaderAndIsr request, we probably can just include 
brokers that are leaders in each LeaderAndIsr request.
                
> Partition.makeFollower() reads broker info from ZK
> --------------------------------------------------
>
>                 Key: KAFKA-575
>                 URL: https://issues.apache.org/jira/browse/KAFKA-575
>             Project: Kafka
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 0.8
>            Reporter: Jun Rao
>            Assignee: Swapnil Ghike
>            Priority: Blocker
>              Labels: bugs
>         Attachments: kafka-575-v1.patch
>
>   Original Estimate: 48h
>  Remaining Estimate: 48h
>
> To follow a new leader, Partition.makeFollower() has to obtain the broker 
> info of the new leader. Currently, it reads that info from ZK for every 
> affected partition. This increases the time for a leader to truly available. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to