[ 
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

        

Reply via email to