Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/2538
@srdo
Thanks for the great work. I'd rather not review the two PRs since @hmcl
already gave a nice review. If you would like to merge twos by yourself, please
replace storm-kafka-client 1.1.
Github user hmcl commented on the issue:
https://github.com/apache/storm/pull/2538
+1.
Once squashed is good to merge as far as I am concerned. Thanks a lot @srdo.
---
Github user srdo commented on the issue:
https://github.com/apache/storm/pull/2538
@hmcl Rebased.
---
Github user hmcl commented on the issue:
https://github.com/apache/storm/pull/2538
@srdo there is a discrepancy between the title of this pull request and the
title of the associated JIRA. What problem are you trying to solve in this
patch? Add meatada, or remove warnings?
If