[ https://issues.apache.org/jira/browse/KAFKA-301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13251734#comment-13251734 ]
Neha Narkhede commented on KAFKA-301: ------------------------------------- I think one thing that can be improved is the structure of our ZK metadata. This is not limited just to this patch. I think for znodes that store more than one value could benefit using a simple json format instead of combinations of "," and ":". This is true for the state change nodes as well as the ISR path. Al though, since this is just a format change and we can still review v2 since the main functionality added is the state change listener, epoch and the ZKQueue. > Implement the broker startup procedure > -------------------------------------- > > Key: KAFKA-301 > URL: https://issues.apache.org/jira/browse/KAFKA-301 > Project: Kafka > Issue Type: Sub-task > Reporter: Neha Narkhede > Assignee: Neha Narkhede > Attachments: kafka-301-draft.patch, kafka-301-v1.patch, > kafka-301-v2.patch > > > This JIRA will involve implementing the list of actions to be taken on broker > startup, as listed by the brokerStartup() and startReplica() algorithm in the > Kafka replication design doc. Since the stateChangeListener is part of > KAFKA-44, this JIRA can leave it as a stub. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira