[GitHub] storm issue #1794: STORM-2193: Fix FilterConfiguration parameter order

2016-11-23 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1794 Great finding. +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 and wishes

[GitHub] storm pull request #1795: STORM-2215: validate blobs are present before subm...

2016-11-23 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1795#discussion_r89416093 --- Diff: storm-core/src/jvm/org/apache/storm/StormSubmitter.java --- @@ -587,4 +592,10 @@ private static double

[GitHub] storm pull request #1795: STORM-2215: validate blobs are present before subm...

2016-11-23 Thread HeartSaVioR
Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1795#discussion_r89416315 --- Diff: storm-core/src/jvm/org/apache/storm/utils/Utils.java --- @@ -1338,6 +1339,26 @@ public static void handleUncaughtException(Throwable t) {

[GitHub] storm issue #1796: STORM-2216: prefer JSONValue.parseWithException

2016-11-23 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1796 +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

[GitHub] storm issue #1783: STORM-2204 Adding caching capabilities in HBaseLookupBolt

2016-11-23 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1783 @revans2 What do you think about this? I'm in favor of adopting Caffeine, and I'm even OK to use Caffeine to master and Guava to 1.x. --- If your project is set up for it, you can reply to this

[GitHub] storm issue #1744: STORM-1276: line for line translation of nimbus to java

2016-11-23 Thread abellina
Github user abellina commented on the issue: https://github.com/apache/storm/pull/1744 nimbus.clj, I made it to mkAssignments. Will continue from there later: - nit: makeBlobCachMap -> makeBlobCacheMap - change: `numDesiredWorkers < numAssignedWorkers` to

FOSDEM 2017 HPC, Bigdata and Data Science DevRoom CFP is closing soon

2016-11-23 Thread Roman Shaposhnik
Hi! apologies for the extra wide distribution (this exhausts my once a year ASF mail-to-all-bigdata-projects quota ;-)) but I wanted to suggest that all of you should consider submitting talks to FOSDEM 2017 HPC, Bigdata and Data Science DevRoom: https://hpc-bigdata-fosdem17.github.io/ It

[GitHub] storm pull request #1789: STORM-2209: Update documents adding new integratio...

2016-11-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1789 --- 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 issue #1789: STORM-2209: Update documents adding new integration for s...

2016-11-23 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1789 Still +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 and wishes so, or if the

[GitHub] storm pull request #1791: STORM-2212: Remove Redundant Declarations in Maven...

2016-11-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1791 --- 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 issue #1791: STORM-2212: Remove Redundant Declarations in Maven POM Fi...

2016-11-23 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1791 +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 and wishes so, or if the feature

[GitHub] storm issue #1792: STORM-2213 ShellSpout has race condition when ShellSpout ...

2016-11-23 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1792 +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,

[GitHub] storm pull request #1790: STORM-2210 - remove array shuffle from ShuffleGrou...

2016-11-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1790 --- 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 #1797: STORM-2210 remove array shuffle from ShuffleGroupi...

2016-11-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1797 --- 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 issue #1797: STORM-2210 remove array shuffle from ShuffleGrouping when...

2016-11-23 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1797 +1, because this is from #1790 that was already approved I will not wait the full 24 hours. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Re: [DISCUSS] Feature Branch for Apache Beam Runner

2016-11-23 Thread Erik Weathers
Hugo, This appears to be the branch: - https://github.com/apache/storm/tree/beam-runner - Erik On Wed, Nov 23, 2016 at 9:34 AM, Hugo Da Cruz Louro wrote: > I somehow missed this email … I would like to contribute to this effort as > well. Please keep me posted. >

Re: [DISCUSS] Feature Branch for Apache Beam Runner

2016-11-23 Thread Hugo Da Cruz Louro
I somehow missed this email … I would like to contribute to this effort as well. Please keep me posted. Thanks. > On Oct 19, 2016, at 8:51 AM, Satish Duggana wrote: > > +1, waiting for that. :) > Currently,there are API changes going on in Beam. It seem they plan to

Should zookeeper be run on the worker machines?

2016-11-23 Thread Tech Id
Hi, Can someone please respond to this zookeeper-for-storm Stack-Overflow question: http://stackoverflow.com/questions/40755137/should-zookeeper-be-run-on-the-worker-machines Thanks TI

[GitHub] storm issue #1790: STORM-2210 - remove array shuffle from ShuffleGrouping wh...

2016-11-23 Thread kevpeek
Github user kevpeek commented on the issue: https://github.com/apache/storm/pull/1790 @HeartSaVioR I created a PR on master with the squashed version of these commits. I hope that's flexible enough to do what you need. https://github.com/apache/storm/pull/1797 --- If your project

[GitHub] storm pull request #1797: STORM-2210 remove array shuffle from ShuffleGroupi...

2016-11-23 Thread kevpeek
GitHub user kevpeek opened a pull request: https://github.com/apache/storm/pull/1797 STORM-2210 remove array shuffle from ShuffleGrouping when counter res… This is a squashed version of PR1790 made against master. You can merge this pull request into a Git repository by running:

Storm sending rate

2016-11-23 Thread Walid Aljoby
Hi everyone, Could anyone has an experience to explain the factors affecting sending rate in Storm? Thank you--RegardsWA

[GitHub] storm issue #1781: STORM-1369: Add MapState implementation to storm-cassandr...

2016-11-23 Thread mkoch1
Github user mkoch1 commented on the issue: https://github.com/apache/storm/pull/1781 That should do it - the batching option has been removed from MapState, in favor of parallel processing with existing opaque and transactional logic to handle consistency. Cassandra batch statements

[GitHub] storm issue #1789: STORM-2209: Update documents adding new integration for s...

2016-11-23 Thread vesense
Github user vesense commented on the issue: https://github.com/apache/storm/pull/1789 Added kubernetes integration. --- 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