[GitHub] storm pull request #1866: STORM-2279: Unable to open bolt page of storm ui

GitHub user raghavgautam opened a pull request: https://github.com/apache/storm/pull/1866 STORM-2279: Unable to open bolt page of storm ui I have added logic to wrap the negative indexes around effectively giving it a python like behaviour. You can merge this pull request into a

[GitHub] storm pull request #1865: Storm 2274 - Adding support for named output strea...

GitHub user roshannaik opened a pull request: https://github.com/apache/storm/pull/1865 Storm 2274 - Adding support for named output stream in HdfsSpout You can merge this pull request into a Git repository by running: $ git pull https://github.com/roshannaik/storm STORM-2274

[GitHub] storm issue #1808: STORM-2225: change spout config to be simpler.

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1808 I think I have addressed all of the review comments so far. I will try to get my 1.x version of the patch up shortly. --- If your project is set up for it, you can reply to this email and have

Re: [DISCUSS] Release Storm 1.1.0

Thanks for the update Jungtaek. I’m verifying the patches now. And they should be mergeable shortly. I think we’ll likely be ready for 1.1.0 and 1.0.3 releases next week. -Taylor > On Jan 6, 2017, at 3:40 AM, Jungtaek Lim wrote: > > I just submitted a patch for STORM-2176

[GitHub] storm issue #1131: STORM-822: Kafka Spout New Consumer API

Github user erikdw commented on the issue: https://github.com/apache/storm/pull/1131 @jianbzhou : you can file a STORM JIRA ticket yourself actually -- you just need to [create an Apache JIRA account](https://issues.apache.org/jira/secure/Signup!default.jspa). --- If your project

[GitHub] storm pull request #1854: STORM-2272: don't leak simulated time

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1854 --- 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 #1785: [STORM-2201] Add dynamic scheduler configuration l...

Github user abellina commented on a diff in the pull request: https://github.com/apache/storm/pull/1785#discussion_r94978694 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/utils/ArtifactoryConfigLoader.java --- @@ -0,0 +1,429 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request #1785: [STORM-2201] Add dynamic scheduler configuration l...

Github user abellina commented on a diff in the pull request: https://github.com/apache/storm/pull/1785#discussion_r94978032 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/resource/ResourceAwareScheduler.java --- @@ -376,18 +388,12 @@ private void initialize(Topologies

[GitHub] storm pull request #1785: [STORM-2201] Add dynamic scheduler configuration l...

Github user abellina commented on a diff in the pull request: https://github.com/apache/storm/pull/1785#discussion_r94978666 --- Diff: storm-core/src/jvm/org/apache/storm/scheduler/utils/ArtifactoryConfigLoader.java --- @@ -0,0 +1,429 @@ +/** + * Licensed to the Apache

[GitHub] storm pull request #1785: [STORM-2201] Add dynamic scheduler configuration l...

Github user abellina commented on a diff in the pull request: https://github.com/apache/storm/pull/1785#discussion_r94975625 --- Diff: pom.xml --- @@ -220,6 +220,7 @@ 0.3.1 1.1.9 0.3.6 +1.2 --- End diff -- why are we

[GitHub] storm issue #1853: STORM-2264 OpaqueTridentKafkaSpout failing after STORM-22...

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1853 I am also OK with reverting STORM-2216 if it is causing a lot of issues. --- 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

[GitHub] storm issue #1853: STORM-2264 OpaqueTridentKafkaSpout failing after STORM-22...

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1853 +1 seems fine 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 #1785: [STORM-2201] Add dynamic scheduler configuration l...

Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/1785#discussion_r94972777 --- Diff: docs/IConfigLoader.md --- @@ -0,0 +1,58 @@ +--- +title: IConfigLoader +layout: documentation +documentation: true +--- +

[GitHub] storm issue #1785: [STORM-2201] Add dynamic scheduler configuration loading.

Github user ppoulosk commented on the issue: https://github.com/apache/storm/pull/1785 @abellina @revans2 Could you look at the re-work on this pull? --- 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

[GitHub] storm issue #1767: STORM-2194: Report error and die, not report error or die

Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1767 @sathyafmt sorry I have taken so long to respond December was a really crazy month for me. From STORM-2194 I see that the SocketTimeoutException goes through the code being changed. The RMI code

[GitHub] storm pull request #1857: STORM-2275: Nimbus crashed during state transition...

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1857 --- 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 #1808: STORM-2225: change spout config to be simpler.

Github user HeartSaVioR commented on a diff in the pull request: https://github.com/apache/storm/pull/1808#discussion_r94917566 --- Diff: external/storm-kafka-client/README.md --- @@ -1,191 +1,3 @@ -#Storm Kafka Spout with New Kafka Consumer API +#Storm Kafka integration

[GitHub] storm pull request #1858: STORM-2276 Remove twitter4j usages due to license ...

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1858 --- 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 #1859: STORM-2276 Remove twitter4j usages due to license ...

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1859 --- 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 #1860: STORM-2276 Remove twitter4j usages due to license ...

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1860 --- 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 #1862: STORM-2278: Allow max number of disruptor queue flusher t...

Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1862 +1 Build failure on storm-core is not related. --- 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 issue #1864: STORM-2176 Workers do not shutdown cleanly and worker hoo...

Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1864 @ptgoetz Could you test my patch to see it can resolve STORM-2176 pretty well? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] storm issue #1864: STORM-2176 Workers do not shutdown cleanly and worker hoo...

Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1864 cherry-picking will work for 1.x and 1.0.x branches. --- 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

Re: [DISCUSS] Release Storm 1.1.0

I just submitted a patch for STORM-2176 . Since it's a small fix, we just need to handle STORM-2228 to release. - Jungtaek Lim (HeartSaVioR) 2017년 1월 5일 (목) 오후 1:43, Jungtaek Lim

[GitHub] storm pull request #1864: STORM-2176 Workers do not shutdown cleanly and wor...

GitHub user HeartSaVioR opened a pull request: https://github.com/apache/storm/pull/1864 STORM-2176 Workers do not shutdown cleanly and worker hooks don't run when a topology is killed * increase supervisor.worker.shutdown.sleep.secs to let workers kill themselves first even stuck

[GitHub] storm pull request #1827: STORM-2243: adds ip address to supervisor id

Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/1827 --- 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 #1131: STORM-822: Kafka Spout New Consumer API

Github user jianbzhou commented on the issue: https://github.com/apache/storm/pull/1131 @erikdw @hmcl , sorry for the late reply. include only part of my fixes is okie for me, we hope finally the community version has the proper fixes so we can use the community version. I don't