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 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 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 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 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