[GitHub] storm pull request: STORM-637: Integrate PartialKeyGrouping into s...

2015-02-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/404 --- 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 enabl

[GitHub] storm pull request: STORM-637: Integrate PartialKeyGrouping into s...

2015-02-03 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/404#issuecomment-72723482 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 enabled

[GitHub] storm pull request: STORM-637: Integrate PartialKeyGrouping into s...

2015-02-02 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/404#issuecomment-72471354 @gdfm I'm happy you like it. --- 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

[GitHub] storm pull request: STORM-637: Integrate PartialKeyGrouping into s...

2015-02-01 Thread gdfm
Github user gdfm commented on the pull request: https://github.com/apache/storm/pull/404#issuecomment-72374010 Looks pretty neat! +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 featu

[GitHub] storm pull request: STORM-637: Integrate PartialKeyGrouping into s...

2015-01-30 Thread revans2
GitHub user revans2 opened a pull request: https://github.com/apache/storm/pull/404 STORM-637: Integrate PartialKeyGrouping into storm API This updates the documentation and provides APIs similar to fieldsGrouping so that it is fairly simple to use. You can merge this pull request