[
https://issues.apache.org/jira/browse/STORM-590?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14587009#comment-14587009
]
ASF GitHub Bot commented on STORM-590:
--------------------------------------
Github user Parth-Brahmbhatt commented on the pull request:
https://github.com/apache/storm/pull/387#issuecomment-112223772
I think we should be able to merge STORM-590 for now given that change is
backward compatible and atleast allows users to move on to Kafka broker APIs.
Personally I just want to re-write the whole kafka connector code.
> KafkaSpout should use kafka consumer api
> ----------------------------------------
>
> Key: STORM-590
> URL: https://issues.apache.org/jira/browse/STORM-590
> Project: Apache Storm
> Issue Type: Improvement
> Components: storm-kafka
> Affects Versions: 0.9.2-incubating
> Reporter: Kai Sasaki
> Assignee: Parth Brahmbhatt
>
> Following below ticket
> https://github.com/apache/storm/pull/338
> KafkaSpout uses kakfa internal data included zk nodes. However it should be
> changed to get these data from kafka consumer api provided kafka project.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
