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

nicu marasoiu edited comment on KAFKA-1549 at 7/26/14 7:46 PM:
---------------------------------------------------------------

Agreed, makes sense - attached patch.
I wonder why I did not receive an email on gmail with your comment, since I am 
subscribed (watching) this issue, do you have any idea why?


was (Author: nmarasoi):
attached

> dead brokers coming in the TopicMetadataResponse
> ------------------------------------------------
>
>                 Key: KAFKA-1549
>                 URL: https://issues.apache.org/jira/browse/KAFKA-1549
>             Project: Kafka
>          Issue Type: Bug
>    Affects Versions: 0.8.2
>         Environment: trunk
>            Reporter: nicu marasoiu
>            Assignee: nicu marasoiu
>         Attachments: bringAllBrokers.patch
>
>
> JunRao confirming my observation that brokers are only added to the 
> metadataCache, never removed: "The way that we update liveBrokers in 
> MetadataCache.updateCache() doesn't seem right. We only add newly received 
> live brokers to the list. However, there could be existing brokers in that 
> list that are now dead. Those dead brokers shouldn't be returned to the 
> clients. We should probably just take the new live broker list and cache it."



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to