GitHub user srdo opened a pull request:
https://github.com/apache/storm/pull/2150
STORM-2541: Fix storm-kafka-client manual subscription not being ableâ¦
⦠to start consuming
I also refactored the manual subscriptions a bit to deduplicate the code. I
think there's no real issue with backporting this to 1.x, because clearly no
one has been using these subscriptions so the API breakage shouldn't bother
anyone.
With regard to testing the manual subscription, I'd like to do that in
https://issues.apache.org/jira/browse/STORM-2542 instead, since it'll make the
current tests use the manual subscription.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/srdo/storm STORM-2541
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2150.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 #2150
----
commit 0a7df0d16bcf98c8e3c12fedfcc7676fe6a68155
Author: Stig Rohde Døssing <[email protected]>
Date: 2017-06-05T12:59:19Z
STORM-2541: Fix storm-kafka-client manual subscription not being able to
start consuming
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
