Github user harshach commented on the issue:
https://github.com/apache/storm/pull/1687
@hmcl also this PR is the same as this one with one more commit?
https://github.com/apache/storm/pull/1757/commits . If so can you close
this one.
---
If your project is set up for it, you can
Github user harshach commented on the issue:
https://github.com/apache/storm/pull/1687
@hmcl these PR titles are very confusing. Can you fix that in both the PRs.
Not sure what it means by "Apache master storm 1694 top storm 2097". Just add
the JIRA title as PR title.
---
If your
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1687
@hmcl OK thanks for following up.
@harshach Please treat my comment as addressed, and also feel free to merge
when you think it's OK to merge.
---
If your project is set up for it, you
Github user hmcl commented on the issue:
https://github.com/apache/storm/pull/1687
@HeartSaVioR I believe I address your valid concerns about the kafka
library versions in this [PR](https://github.com/apache/storm/pull/1757)
---
If your project is set up for it, you can reply to
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/1687
Since it does load the two modules - storm-kafka and storm-kafka-client -
but only load kafka-client with `${storm.kafka.client.version}`. Does
storm-starter work with both storm-kafka and
