You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Jay Kreps (JIRA)" <ji...@apache.org> on 2013/02/11 21:03:12 UTC

[jira] [Commented] (KAFKA-755) standardizing json values stored in ZK

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

Jay Kreps commented on KAFKA-755:
---------------------------------

Also
c. Think through how any generalization would work (e.g. (1) can't really be extended beyond partitions because the partitions are at the top level of the object.
d. Standardize capitalization: we have dashes, underscores, camel case and pretty much everything else
e: Document them and going forward add to the documentation. This will ensure future additions are thought through holistically with all the other structures.
                
> standardizing json values stored in ZK
> --------------------------------------
>
>                 Key: KAFKA-755
>                 URL: https://issues.apache.org/jira/browse/KAFKA-755
>             Project: Kafka
>          Issue Type: Bug
>          Components: core
>            Reporter: Jun Rao
>            Priority: Blocker
>
> Currently, we have the following paths in ZK that stores non-singleton values.
> 1. Topic assignment value:
> /brokers/topics/topic
> { "0": ["0"] }
> 2. LeaderAndISR info:
> /brokers/topics/test/partitions/0/leaderAndISR
> { "ISR":"0","leader":"0","controllerEpoch":"1","leaderEpoch":"0" }
> 3. broker registration:
> /brokers/ids/0
> 192.168.1.148:9092:9999
> 4. partition reassignment path
> It would be good if we do the following:
> a. make them true json (e.g., using number as the value for broker/partition, instead of string).
> b. add version support for future growth.

--
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