[GitHub] storm pull request #1718: BackStorm 2124 show requested cpu mem for each com...

GitHub user abellina opened a pull request: https://github.com/apache/storm/pull/1718 BackStorm 2124 show requested cpu mem for each component 1x Backport of https://github.com/apache/storm/pull/1713 You can merge this pull request into a Git repository by running: $ git pull h

[GitHub] storm issue #1716: STORM-2126: fix NPE due to race condition in compute-new-...

Github user abellina commented on the issue: https://github.com/apache/storm/pull/1716 @revans2 this fixes RAT issue. --- 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 pull request #1692: Fix STORM-2017

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1692 --- 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 enab

Error while building locally

Hello Devs, I am trying to build Storm locally. I am getting following error. I am using Windows 8, maven 3.3.9 and java 1.8 Exception Trace --- [INFO] Error stacktraces are turne

[GitHub] storm issue #1716: STORM-2126: fix NPE due to race condition in compute-new-...

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1716 +1 looks good to me --- 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,

Re: Error while building locally

It looks like maven somehow does not have permission to write to  C:\Users\sonu\.m2\repository\  I am not a windows expert, but I thought maven stored the local repository on windows under C:\Documents and Settings\{your-username}\.m2  It seems like you have an odd maven build, something odd

Re: [DISCUSS] Plan for merge SQE and Storm SQL

If there are no objections, I’d like to proceed with this approach. -Taylor > On Sep 27, 2016, at 2:22 PM, P. Taylor Goetz wrote: > > Indeed it does not compile against any version due to the missing changes to > storm-kafka and storm-redis that Morrigan pointed out. > > So for now I’m leani

[GitHub] storm pull request #1691: STORM-2090: Add integration test for storm windowi...

Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1691#discussion_r80987898 --- Diff: integration-test/README.md --- @@ -0,0 +1,59 @@ +End to end storm integration tests +== + +Running tests

[GitHub] storm issue #1691: STORM-2090: Add integration test for storm windowing

Github user hmcl commented on the issue: https://github.com/apache/storm/pull/1691 @raghavgautam I am +1 overall. I believe it makes sense to have these integration tests run in the mvn integration-test phase, but that's the only detail I have to add. +1 --- If your project

[GitHub] storm pull request #1691: STORM-2090: Add integration test for storm windowi...

Github user hmcl commented on a diff in the pull request: https://github.com/apache/storm/pull/1691#discussion_r80988759 --- Diff: integration-test/README.md --- @@ -0,0 +1,59 @@ +End to end storm integration tests +== + +Running tests

Re: [DISCUSS] Plan for merge SQE and Storm SQL

The changes are available in GitHub, I just overlooked it. And they’re actually neatly contained in two commits: The changes to storm-kafka can be found here: https://github.com/jwplayer/storm/commit/2069c76695a225e4bb8f402c89e572836104755a

[GitHub] storm pull request #1719: [STORM-1057] Add throughput metrics to spouts/bolt...

GitHub user wangli1426 opened a pull request: https://github.com/apache/storm/pull/1719 [STORM-1057] Add throughput metrics to spouts/bolts and display them on web ui for master branch Hi @HeartSaVioR, @revans2, @harshach,@d2r,@unsleepy22, In this PR, throughput of tasks is

[GitHub] storm issue #753: [STORM-1057] Add throughput metrics to spouts/bolts and di...

Github user wangli1426 commented on the issue: https://github.com/apache/storm/pull/753 This new feature has been implemented on master branch in #1719. Please review. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

Re: Error while building locally

I could able to resolve this problem and could able to build storm locally. Now when I try to package, it hangs. --- c:/storm/storm-dist\binary> mvn package [INFO] Sca