GitHub user shangan opened a pull request:

    https://github.com/apache/kafka/pull/329

    KAFKA-2146. adding partition did not find the correct startIndex

    TopicCommand provide a tool to add partitions for existing topics. It try 
to find the startIndex from existing partitions. There's a minor flaw in this 
process, it try to use the first partition fetched from zookeeper as the start 
partition, and use the first replica id in this partition as the startIndex.
    One thing, the first partition fetched from zookeeper is not necessary to 
be the start partition. As partition id begin from zero, we should use 
partition with id zero as the start partition.
    The other, broker id does not necessary begin from 0, so the startIndex is 
not necessary to be the first replica id in the start partition.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/shangan/kafka trunk-KAFKA-2146

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/kafka/pull/329.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #329
    
----
commit 8937cf75240bf48c1b70c1c2be461c5577dba3ac
Author: chenshangan <chenshan...@meituan.com>
Date:   2015-10-19T13:06:24Z

    KAFKA-2146. adding partition did not find the correct startIndex

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to