[GitHub] storm issue #1789: STORM-2209: Update documents adding new integration for s...

2016-11-28 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1789 I created https://github.com/apache/storm/pull/1802 for 1.x-branch. --- 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

[GitHub] storm issue #1789: STORM-2209: Update documents adding new integration for s...

2016-11-24 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1789 @revans2 @HeartSaVioR I think this PR should be cherry-picked to 1.x-branch since 1.x has contained these new modules . And we'd better sync documents to storm site. --- If your project is set up

[GitHub] storm issue #1789: STORM-2209: Update documents adding new integration for s...

2016-11-23 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1789 Still +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

[GitHub] storm issue #1789: STORM-2209: Update documents adding new integration for s...

2016-11-23 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1789 Added kubernetes integration. --- 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

[GitHub] storm issue #1789: STORM-2209: Update documents adding new integration for s...

2016-11-22 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1789 @revans2 @HeartSaVioR Yes, the new files are copied from external modules README.md. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] storm issue #1789: STORM-2209: Update documents adding new integration for s...

2016-11-22 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1789 +1 I think @vesense copied the content from README.md for each of external modules. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] storm issue #1789: STORM-2209: Update documents adding new integration for s...

2016-11-21 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1789 +1 looks good to me. Although I am not familiar with all of the bolts, spouts, etc listed here to be sure that the code in the examples is correct. --- If your project is set up for it, you can