[
https://issues.apache.org/jira/browse/KAFKA-3525?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15233241#comment-15233241
]
Manikumar Reddy commented on KAFKA-3525:
----------------------------------------
Yes, by looking this function alone it is possible. But, the create part of
the code will never get execute (we create all the required zk paths during
kafka startup process). May be we can just cleanup code.
> max.reserved.broker.id off-by-one error
> ---------------------------------------
>
> Key: KAFKA-3525
> URL: https://issues.apache.org/jira/browse/KAFKA-3525
> Project: Kafka
> Issue Type: Bug
> Components: config
> Reporter: Alan Braithwaite
> Assignee: Manikumar Reddy
> Priority: Blocker
> Fix For: 0.10.0.0
>
>
> There's an off-by-one error in the config check / id generation for
> max.reserved.broker.id setting. The auto-generation will generate
> max.reserved.broker.id as the initial broker id as it's currently written.
> Not sure what the consequences of this are if there's already a broker with
> that id as I didn't test that behavior.
> This can return 0 + max.reserved.broker.id:
> https://github.com/apache/kafka/blob/8dbd688b1617968329087317fa6bde8b8df0392e/core/src/main/scala/kafka/utils/ZkUtils.scala#L213-L215
> However, this does a <= check, which is inclusive of max.reserved.broker.id:
> https://github.com/apache/kafka/blob/8dbd688b1617968329087317fa6bde8b8df0392e/core/src/main/scala/kafka/server/KafkaConfig.scala#L984-L986
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
