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

ASF GitHub Bot commented on STORM-1065:
---------------------------------------

GitHub user lujinhong opened a pull request:

    https://github.com/apache/storm/pull/1140

     [STORM-1065] kafka-partition can not find leader in zookeeper #1132 

    

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

    $ git pull https://github.com/lujinhong/storm master

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

    https://github.com/apache/storm/pull/1140.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 #1140
    
----
commit f01f0c1953500d2b23b5f0b6c0fa1083a46a984f
Author: lujinhong <lujinho...@gmail.com>
Date:   2016-02-23T10:23:23Z

    Update KafkaUtils.java
    
    add checkLeader() to check if the leader is active.

commit ad610ee71149f793f986e6ad8085f1370c1317c0
Author: lujinhong <lujinho...@gmail.com>
Date:   2016-02-23T10:25:22Z

    Update ZkCoordinator.java
    
    if leader of the patition is not active, do not add to _manager.

commit 020ef1d1cf87fe19ca0e26f3043bf05d0d4f95b7
Author: lujinhong <lujinho...@gmail.com>
Date:   2016-02-23T10:30:17Z

    Update PartitionManager.java

----


> storm-kafka : kafka-partition can not find leader in zookeeper 
> ---------------------------------------------------------------
>
>                 Key: STORM-1065
>                 URL: https://issues.apache.org/jira/browse/STORM-1065
>             Project: Apache Storm
>          Issue Type: Improvement
>          Components: storm-kafka
>            Reporter: dongxinwang
>            Priority: Minor
>
> If the Kafka cluster is not consistent with the zookeeper data, the partition 
> can not find leader in zookeeper.In storm-kafka, it throws runtime 
> exception"No leader found for partition",it is not friendly.
> Suggestion:
> If there is no leader partition in zookeeper,don't add the partition to 
> GlobalPartitionInformation object,instead of throwing runtime exception.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to