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