[GitHub] storm pull request: [STORM-1513] Added documentation for filter, m...

2016-02-04 Thread HeartSaVioR
Github user HeartSaVioR commented on the pull request: https://github.com/apache/storm/pull/1070#issuecomment-179794733 +1 Great --- 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

[GitHub] storm pull request: [STORM-1513] Added documentation for filter, m...

2016-02-04 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1070 --- 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

[GitHub] storm pull request: [STORM-1513] Added documentation for filter, m...

2016-02-04 Thread satishd
Github user satishd commented on the pull request: https://github.com/apache/storm/pull/1070#issuecomment-179910533 +1 LGTM --- 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

[GitHub] storm pull request: [STORM-1513] Added documentation for filter, m...

2016-02-03 Thread arunmahadevan
GitHub user arunmahadevan opened a pull request: https://github.com/apache/storm/pull/1070 [STORM-1513] Added documentation for filter, map and flatMap You can merge this pull request into a Git repository by running: $ git pull https://github.com/arunmahadevan/storm

[GitHub] storm pull request: [STORM-1513] Added documentation for filter, m...

2016-02-03 Thread arunmahadevan
Github user arunmahadevan commented on a diff in the pull request: https://github.com/apache/storm/pull/1070#discussion_r51727059 --- Diff: documentation/Trident-API-Overview.md --- @@ -77,13 +77,67 @@ Now suppose you had these tuples with fields ["a", "b", "c"]: If you ran

[GitHub] storm pull request: [STORM-1513] Added documentation for filter, m...

2016-02-03 Thread redsanket
Github user redsanket commented on a diff in the pull request: https://github.com/apache/storm/pull/1070#discussion_r51726012 --- Diff: documentation/Trident-API-Overview.md --- @@ -77,13 +77,67 @@ Now suppose you had these tuples with fields ["a", "b", "c"]: If you ran this

[GitHub] storm pull request: [STORM-1513] Added documentation for filter, m...

2016-02-03 Thread redsanket
Github user redsanket commented on the pull request: https://github.com/apache/storm/pull/1070#issuecomment-179408181 LGTM +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