Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/2018
@ptgoetz
OK either applying or not applying this to 1.0.x is fine for me.
Given that you're planning to apply this to master branch as well, I think
this is mergeable now.
+1
---
Github user ptgoetz commented on the issue:
https://github.com/apache/storm/pull/2018
@HeartSaVioR Yes, that's my plan. I prioritized the 1.1.x branch since it's
needed for the 1.1.0 release.
I'm not sure we want to apply it to the 1.0.x branch since we have a number
of
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/2018
@ptgoetz
Could you make the change for master branch? Maybe 1.0.x branch too.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user HeartSaVioR commented on the issue:
https://github.com/apache/storm/pull/2018
Looks great otherwise. I'd be +1 once build issue is fixed.
---
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
Github user vesense commented on the issue:
https://github.com/apache/storm/pull/2018
+1
---
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