[GitHub] storm pull request: Update pom.xml

2016-01-05 Thread vesense
Github user vesense commented on the pull request: https://github.com/apache/storm/pull/599#issuecomment-169224710 Sorry I took so long to respond. It is convenient for multi-lang user to use FLUX just in one jar. As a minor issue, now, It is OK for me. In the past, I wanted

[GitHub] storm pull request: Update pom.xml

2016-01-04 Thread ptgoetz
Github user ptgoetz commented on the pull request: https://github.com/apache/storm/pull/599#issuecomment-168829281 -1 flux-wrappers add convenience multi-lang spout/bolt implementations. Arguably, they could be included in flux-core. I'd rather go that route than set flux-wrappers to

[GitHub] storm pull request: Update pom.xml

2015-12-28 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/storm/pull/599#issuecomment-167627569 @vesense, could you create a Jira issue for this change and add it to the title of this pull request? What problem is the current scope causing here that needs to be

[GitHub] storm pull request: Update pom.xml

2015-06-19 Thread vesense
GitHub user vesense opened a pull request: https://github.com/apache/storm/pull/599 Update pom.xml add flux-wrappers scope You can merge this pull request into a Git repository by running: $ git pull https://github.com/vesense/storm patch-1 Alternatively you can review and