[ANNOUNCE] Apache Storm 2.2.0 Released

The Apache Storm community is pleased to announce the release of Apache Storm version 2.2.0. Apache Storm is a distributed, fault-tolerant, and high-performance realtime computation system that provides strong guarantees on the processing of data. You can read more about Storm on the project

Re: [VOTE] Release Apache Storm 2.2.0 (RC2)

ty works. > > -Kishor > > On Thu, Jun 18, 2020 at 2:18 PM Govind Menon wrote: > > > This is a call to vote on releasing Apache Storm 2.2.0 (As a personal > note > > I would like to thank P. Taylor Goetz and Ethan Li for all the > > documentation they have set

Re: [VOTE] Release Apache Storm 2.2.0 (RC2)

orm.git;a=commit;h=bf1986345de5de605abb4fc7b6051fce762bbca5 > > > > > > On Jun 18, 2020, at 1:18 PM, Govind Menon wrote: > > > > This is a call to vote on releasing Apache Storm 2.2.0 (As a personal > note > > I would like to thank P. Taylor Goetz and Ethan Li for all

Re: Adding arm64 support to travis-ci in storm

jobs/352331819 > I am guessing that the code is making some assumptions that are not working > for arm64 travis job. > Can you please review the travis logs? > > Regards > Rahul > > > > > On Mon, Jun 22, 2020 at 8:29 PM Govind Menon > wrote: > > >

Re: Adding arm64 support to travis-ci in storm

What is the Python version which is showing the error? It's possible your default Python is also 3 - try pip2 install mock as well. On Mon, 22 Jun 2020 at 04:26, Rahul Aggarwal < rahul.aggar...@puresoftware.com> wrote: > Hi Team, > > I am adding arm64 support to storm's travis-ci with jdk 8 and

[VOTE] Release Apache Storm 2.2.0 (RC2)

This is a call to vote on releasing Apache Storm 2.2.0 (As a personal note I would like to thank P. Taylor Goetz and Ethan Li for all the documentation they have set up to make releasing easier) Full list of changes in this release:

Re: [VOTE] Release Apache Storm 2.2.0 (RC1)

voke0(Native Method) > > > > -Kishor > > > > > > On Mon, Jun 15, 2020 at 1:55 AM Julien Nioche < > lists.digitalpeb...@gmail.com> > > wrote: > > > >> +1 (non binding) > >> > >> I have been running a crawl with StormCrawl

[VOTE] Release Apache Storm 2.2.0 (RC1)

This is a call to vote on releasing Apache Storm 2.2.0 (As a personal note I would like to thank P. Taylor Goetz and Ethan Li for all the documentation they have set up to make releasing easier) Full list of changes in this release:

Re: [VOTE] Release Apache Storm 2.2.0

gt; >> > > > >> I have tried to get StormCrawler to work with Storm 2.x but the > issues > > > >> above have prevented me from doing so. At least one other person has > > had > > > >> the same problems as you can see in the JIRAs above.

[VOTE] Release Apache Storm 2.2.0

This is a call to vote on releasing Apache Storm 2.2.0 (As a personal note I would like to thank P. Taylor Goetz and Ethan Li for all the documentation they have set up to make releasing easier) Full list of changes in this release:

Re: [VOTE] Release Apache Storm 2.0.0 (rc4)

+1 for continuing with the RC On Tue, 29 Jan 2019 at 16:49, Arun Mahadevan wrote: > Agree with continuing with the RC if this is not a blocker. > > On Tue, 29 Jan 2019 at 12:29, Roshan Naik > wrote: > > > Correct me if I am wrong, this seems to be a bug with a workaround but > > not an

Re: [VOTE] Migration of git repos to gitbox.apache.org

+1 On Fri, 18 Jan 2019 at 13:33, Stig Rohde Døssing wrote: > +1 > > Den fre. 18. jan. 2019 kl. 16.59 skrev Kishorkumar Patil > : > > > +1 > > > > On Fri, Jan 18, 2019 at 10:47 AM Ethan Li > > wrote: > > > > > This is a call to vote on migration of git repos to gitbox.apache.org > < > > >

[GitHub] storm issue #2930: STORM-3274: Migrates storm CLI to using argparse making d...

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2930 @srdo I've addressed the 4 comments you had - thank you for the review ---

[GitHub] storm pull request #2930: STORM-3274: Migrates storm CLI to using argparse m...

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2930#discussion_r247197760 --- Diff: bin/storm.py --- @@ -296,787 +239,1044 @@ def exec_storm_class(klass, jvmtype="-server", jvmopts=[], extrajars=[], args=[]

[GitHub] storm issue #2930: STORM-3274: Migrates storm CLI to using argparse making d...

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2930 @srdo Addressed most of your comments - Tests are run for python2 and python 3 - Help is sorted alphabetically now - running just storm prints the entire help

Re: New Storm Committer and PMC Member

gt; > > > > > > > Congratulations! Govind. Well deserved! > > > > > > > > > > > > Ethan > > > > > > > > > > > > > On Jan 9, 2019, at 12:40 PM, Hugo Louro > > > wrote: > > > > > >

[GitHub] storm pull request #2930: STORM-3274: Migrates storm CLI to using argparse m...

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2930#discussion_r245726485 --- Diff: storm-client/pom.xml --- @@ -240,6 +240,29

[GitHub] storm pull request #2930: STORM-3274: Migrates storm CLI to using argparse m...

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2930#discussion_r245709584 --- Diff: storm-client/pom.xml --- @@ -240,6 +240,29

Re: Storm 2.0.0 release?

+1 On Jan 4, 2019 23:46, "Roshan Naik" wrote: +1 Sent from Yahoo Mail for iPhone On Friday, January 4, 2019, 9:02 PM, Ethan Li wrote: +1 on this Ethan Li > On Jan 4, 2019, at 20:19, P. Taylor Goetz wrote: > > If no one objects, I’ll kick off a release candidate. > > -Taylor > >> On

[GitHub] storm issue #2932: STORM-3274: Adds mock to Travis pipeline so it can run Py...

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2932 @srdo Will do ---

[GitHub] storm pull request #2932: STORM-3274: Adds mock to Travis pipeline so it can...

Github user govind-menon closed the pull request at: https://github.com/apache/storm/pull/2932 ---

[GitHub] storm pull request #2932: STORM-3274: Adds mock to Travis pipeline so it can...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2932 STORM-3274: Adds mock to Travis pipeline so it can run Python tests f… …or cli migration You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] storm pull request #2931: STORM-3307: Fixes error time on UI for time of las...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2931 STORM-3307: Fixes error time on UI for time of last error on componen… …t page You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind

[GitHub] storm issue #2930: STORM-3274: Migrates storm CLI to using argparse making d...

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2930 Tests will be forthcoming ---

[GitHub] storm pull request #2930: STORM-3274: Migrates storm CLI to using argparse m...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2930 STORM-3274: Migrates storm CLI to using argparse making documentation… … more accessible You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

GitHub user govind-menon reopened a pull request: https://github.com/apache/storm/pull/2881 STORM-3259: NUMA Support for Storm Only functional changes - putting up for review now and tests soon to follow. Have Done the following tests 1. Mixed cluster - numa

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon closed the pull request at: https://github.com/apache/storm/pull/2881 ---

[GitHub] storm pull request #2914: STORM-1311: Removing superfluous debugging

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2914 STORM-1311: Removing superfluous debugging You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm patch-2 Alternatively you

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r229883970 --- Diff: storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java --- @@ -414,6 +418,34 @@ public void validateInteger(String name

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r229869065 --- Diff: storm-client/src/jvm/org/apache/storm/daemon/worker/WorkerState.java --- @@ -622,22 +624,26 @@ public boolean areAllConnectionsReady

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r229854058 --- Diff: storm-server/src/test/java/org/apache/storm/daemon/supervisor/BasicContainerTest.java --- @@ -319,7 +319,15 @@ public void testLaunch

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r229853904 --- Diff: storm-server/src/main/java/org/apache/storm/scheduler/resource/normalization/NormalizedResourceOffer.java --- @@ -81,6 +81,20 @@ public void

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r227413704 --- Diff: storm-client/src/jvm/org/apache/storm/Config.java --- @@ -1041,6 +1041,19 @@ @isPositiveNumber @NotNull public

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r227146299 --- Diff: storm-client/src/jvm/org/apache/storm/daemon/worker/Worker.java --- @@ -348,14 +350,27 @@ public void doExecutorHeartbeats

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r226082482 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -121,11 +121,68 @@ private static String memoizedLocalHostnameString

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r226082082 --- Diff: storm-client/src/jvm/org/apache/storm/daemon/worker/WorkerState.java --- @@ -630,7 +632,7 @@ public boolean areAllConnectionsReady

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r226081834 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -121,11 +121,68 @@ private static String memoizedLocalHostnameString

[GitHub] storm issue #2881: STORM-3259: NUMA Support for Storm

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2881 Includes work by @kishorvpatil ---

[GitHub] storm pull request #2881: STORM-3259: NUMA Support for Storm

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2881 STORM-3259: NUMA Support for Storm Only functional changes - putting up for review now and tests soon to follow. Have Done the following tests 1. Mixed cluster - numa

[GitHub] storm pull request #2854: STORM-3239: Adding dumpjstack action and removing ...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2854 STORM-3239: Adding dumpjstack action and removing wrong timeout param… …eter You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind

Re: [VOTE] Release Apache Storm 2.0.0 (rc1)

> >> > > > >> -Taylor > > > >> > > > >>> On Sep 25, 2018, at 1:56 PM, Bobby Evans wrote: > > > >>> > > > >>> Just FYI Taylor I just merged in 3 JIRA to the master branch and > will > > >

[GitHub] storm pull request #2845: STORM-3234: Replace old metrics docs with better d...

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2845#discussion_r220219980 --- Diff: docs/ClusterMetrics.md --- @@ -0,0 +1,256 @@ +--- +title: Cluster Metrics +layout: documentation +documentation: true

[GitHub] storm pull request #2845: STORM-3234: Replace old metrics docs with better d...

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2845#discussion_r220219222 --- Diff: docs/ClusterMetrics.md --- @@ -0,0 +1,256 @@ +--- +title: Cluster Metrics +layout: documentation +documentation: true

Re: [VOTE] Release Apache Storm 2.0.0 (rc1)

Hi all, I introduced a bug with the visualization as part of the UI migration with the following fix - https://github.com/apache/storm/pull/2849. It's not a show stopper by any means and there's a workaround (specifically hiding system stats on the UI). I think we should include this in the RC.

[GitHub] storm pull request #2849: STORM-1311: Fix for visualization when sys metrics...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2849 STORM-1311: Fix for visualization when sys metrics aren't present yet You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm

Re: Regarding releasing Apache Storm 2.0.0

STORM-3217 and STORM-3221 have been fixed - +1 from me for 2.0 RC. On Wed, Sep 12, 2018 at 10:01 AM Govind Menon wrote: > Hi all, > > There are some regressions that I introduced as part of STORM-1311 which > I'm working on as part of https://issues.apache.org/jira/browse/STORM-3217

[GitHub] storm pull request #2828: STORM-3217: Fixing getComponentPage API call

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2828 STORM-3217: Fixing getComponentPage API call You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm STORM-3217 Alternatively

[GitHub] storm pull request #2827: STORM-3221: Fixing utilization in clusterSummary A...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2827 STORM-3221: Fixing utilization in clusterSummary API call You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm STORM-3221

Re: Regarding releasing Apache Storm 2.0.0

Hi all, There are some regressions that I introduced as part of STORM-1311 which I'm working on as part of https://issues.apache.org/jira/browse/STORM-3217 and https://issues.apache.org/jira/browse/STORM-3221. These should be fixed before a 2.x release I have code working on the Yahoo internal

[GitHub] storm pull request #2817: STORM-1311: Fixing using storm conf instead of top...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2817 STORM-1311: Fixing using storm conf instead of topo conf for UI ACL c… …heck You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind

[GitHub] storm pull request #2815: STORM-1311: Missed UIFilter Plugin setup

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2815 STORM-1311: Missed UIFilter Plugin setup You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm master Alternatively you can

[GitHub] storm issue #2799: STORM-3183: Fix for visualization on Storm API

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2799 @HeartSaVioR Fixed the component page. ---

[GitHub] storm pull request #2799: STORM-3183: Fix for visualization on Storm API

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2799 STORM-3183: Fix for visualization on Storm API You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm STORM-3183

[GitHub] storm pull request #2797: STORM-3179: Fixing getNimbusSummary in Storm API

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2797 STORM-3179: Fixing getNimbusSummary in Storm API You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm STORM-3179

[GitHub] storm issue #2796: STORM-3181: Fix for completeLatency in TopologyStats

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2796 @HeartSaVioR 2 left. ---

[GitHub] storm pull request #2796: STORM-3181: Fix for completeLatency in TopologySta...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2796 STORM-3181: Fix for completeLatency in TopologyStats You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm STORM-3181

[GitHub] storm issue #2795: STORM-3182: Removing superfluous topo id check from owner...

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2795 @HeartSaVioR Fix for 3182. Working on the others. ---

[GitHub] storm pull request #2795: STORM-3182: Removing superfluous topo id check fro...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2795 STORM-3182: Removing superfluous topo id check from owner resource AP… …I request You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] storm issue #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2752 @HeartSaVioR @revans2 I've done the changes you requested. Please let me know of any other changes. ---

[GitHub] storm issue #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2752 @HeartSaVioR I addressed your review comments. Currently fixing checkstyle. ---

[GitHub] storm pull request #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2752#discussion_r205816093 --- Diff: pom.xml --- @@ -329,7 +329,7 @@ 3.1.0 1.0 0.13.1 -2.24.1 +2.27 --- End diff

[GitHub] storm pull request #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2752#discussion_r205815906 --- Diff: storm-core/src/jvm/org/apache/storm/ui/filters/AuthorizedUserFilter.java --- @@ -0,0 +1,137 @@ +/* + * Licensed to the Apache

[GitHub] storm pull request #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2752#discussion_r205815985 --- Diff: storm-core/src/jvm/org/apache/storm/utils/Monitor.java --- @@ -1,12 +1,18 @@ /** - * Licensed to the Apache Software Foundation (ASF

[GitHub] storm pull request #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2752#discussion_r201717965 --- Diff: storm-core/src/jvm/org/apache/storm/ui/UIHelpers.java --- @@ -295,9 +391,1104 @@ public static String getJsonResponseBody(Object data, String

[GitHub] storm pull request #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2752#discussion_r201717748 --- Diff: storm-core/test/jvm/org/apache/storm/ui/StormApiResourceTest.java --- @@ -0,0 +1,19 @@ +package org.apache.storm.ui; + +import

[GitHub] storm pull request #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2752#discussion_r201717619 --- Diff: storm-core/src/jvm/org/apache/storm/ui/filters/AuthorizedUserFilter.java --- @@ -0,0 +1,162 @@ +/* + * Licensed to the Apache

[GitHub] storm pull request #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2752#discussion_r201717451 --- Diff: storm-client/src/jvm/org/apache/storm/Constants.java --- @@ -55,5 +55,7 @@ public static final String

[GitHub] storm pull request #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2752#discussion_r201717347 --- Diff: storm-core/src/jvm/org/apache/storm/ui/UIServer.java --- @@ -0,0 +1,101 @@ +/* + * Licensed to the Apache Software Foundation (ASF

[GitHub] storm issue #2752: Storm 1311 Migration of UI from clj to Java

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2752 I've addressed the feedback in the new commit - currently checkstyle is failing (which is mostly Javadocs) - should have that up soon ---

[GitHub] storm pull request #2752: Storm 1311 Migration of UI from clj to Java

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2752 Storm 1311 Migration of UI from clj to Java This PR basically duplicates the entire UI core in Java as an embedded Jetty Server. The follow on PR will delete the clj code and substitute

[GitHub] storm issue #2738: Storm 1311 Migration of UI from Clojure to Jersey/Grizzly...

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2738 PR is missing a lot of changes - will put up again in some time. ---

[GitHub] storm pull request #2738: Storm 1311 Migration of UI from Clojure to Jersey/...

Github user govind-menon closed the pull request at: https://github.com/apache/storm/pull/2738 ---

[GitHub] storm pull request #2658: Storm 2171: Removing LocalFsBlobStore specific Nim...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2658 Storm 2171: Removing LocalFsBlobStore specific Nimbus actions You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm STORM

Re: New Committer/PMC Member: Ethan Li

Congratulations Ethan! On Mon, Apr 16, 2018 at 11:58 AM, Roshan Naik wrote: > Congratulations Ethan. > > > Sent from Yahoo Mail for iPhone > > > On Monday, April 16, 2018, 9:23 AM, Bobby Evans wrote: > > Please Join with me in welcoming Ethan

[GitHub] storm pull request #2312: YSTORM-2725: Generic Resource Scheduling - initial...

Github user govind-menon closed the pull request at: https://github.com/apache/storm/pull/2312 ---

[GitHub] storm issue #2491: STORM-2872: TestRebalance can be flaky - fixing

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2491 @HeartSaVioR May this be merged? ---

[GitHub] storm issue #2490: STORM-2840: Getting resource availability from pre-GRAS s...

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2490 @HeartSaVioR May this be merged? ---

[GitHub] storm pull request #2491: STORM-2872: TestRebalance can be flaky - fixing

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2491 STORM-2872: TestRebalance can be flaky - fixing You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm YSTORM-4457-II

[GitHub] storm pull request #2490: STORM-2840: Getting resource availability from pre...

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2490 STORM-2840: Getting resource availability from pre-GRAS supervisors You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm

[GitHub] storm issue #2456: STORM-2872: Fix for wouldFit and rebalance as part of Gen...

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2456 @HeartSaVioR Done. As an aside the test failure appears to be on the master branch as well. ---

[GitHub] storm issue #2456: STORM-2872: Fix for wouldFit and rebalance as part of Gen...

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2456 @HeartSaVioR apologies for lack of description. I filed a new subtask and added a description ---

[GitHub] storm pull request #2456: STORM-2725: Fix for wouldFit and rebalance as part...

GitHub user govind-menon reopened a pull request: https://github.com/apache/storm/pull/2456 STORM-2725: Fix for wouldFit and rebalance as part of GenericResourceAwareScheduling changes You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] storm pull request #2456: STORM-2725: Fix for wouldFit and rebalance as part...

Github user govind-menon closed the pull request at: https://github.com/apache/storm/pull/2456 ---

[GitHub] storm pull request #2456: YSTORM-4457: Fix for wouldFit

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2456 YSTORM-4457: Fix for wouldFit You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm YSTORM-4457-II Alternatively you can

[GitHub] storm issue #2437: [STORM-2834] getOwnerResourceSummaries not working proper...

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2437 @revans2 There is now - https://issues.apache.org/jira/browse/STORM-2840 ---

[GitHub] storm pull request #2410: STORM-2808: Fix for NPE in UI

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2410 STORM-2808: Fix for NPE in UI You can merge this pull request into a Git repository by running: $ git pull https://github.com/govind-menon/storm STORM-2808 Alternatively you can review

[GitHub] storm issue #2385: STORM-2727: Generic Resource Aware Scheduling

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2385 @HeartSaVioR Thank you - I will change the commit and PR titles. ---

[GitHub] storm issue #2385: STORM-2727: Generic Resource Aware Scheduling

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2385 Thanks @revans2 ! @HeartSaVioR could you take a look? ---

[GitHub] storm pull request #2385: STORM-2727: Generic Resource Aware Scheduling

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2385#discussion_r149523415 --- Diff: storm-server/src/main/java/org/apache/storm/scheduler/resource/strategies/scheduling/DefaultResourceAwareStrategy.java --- @@ -18,93 +18,58

[GitHub] storm pull request #2385: STORM-2727: Generic Resource Aware Scheduling

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2385#discussion_r149511408 --- Diff: storm-client/src/storm.thrift --- @@ -496,6 +496,8 @@ struct WorkerResources { 3: optional double cpu; 4: optional double

[GitHub] storm pull request #2385: STORM-2727: Generic Resource Aware Scheduling

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2385#discussion_r149225052 --- Diff: examples/storm-loadgen/src/main/java/org/apache/storm/loadgen/CaptureLoad.java --- @@ -430,37 +427,39 @@ public static void main(String

[GitHub] storm issue #2385: STORM-2727: Generic Resource Aware Scheduling

Github user govind-menon commented on the issue: https://github.com/apache/storm/pull/2385 @HeartSaVioR this PR should cover STORM-2725 (which covers STORM-2727). I apologize for the confusion - I was planning this as several PRs and it snowballed into one big PR. ---

[GitHub] storm pull request #2385: STORM-2727: Generic Resource Aware Scheduling

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2385#discussion_r148045890 --- Diff: storm-client/src/jvm/org/apache/storm/scheduler/SupervisorDetails.java --- @@ -17,12 +17,10 @@ */ package

[GitHub] storm pull request #2385: STORM-2727: Generic Resource Aware Scheduling

GitHub user govind-menon reopened a pull request: https://github.com/apache/storm/pull/2385 STORM-2727: Generic Resource Aware Scheduling Remaining 1. Add Tests 2. Do more manual testing You can merge this pull request into a Git repository by running: $ git pull

[GitHub] storm pull request #2385: STORM-2727: Generic Resource Aware Scheduling

Github user govind-menon closed the pull request at: https://github.com/apache/storm/pull/2385 ---

[GitHub] storm pull request #2385: YSTORM-2727: Generic Resource Aware Scheduling

GitHub user govind-menon reopened a pull request: https://github.com/apache/storm/pull/2385 YSTORM-2727: Generic Resource Aware Scheduling Remaining 1. Add Tests 2. Do more manual testing You can merge this pull request into a Git repository by running: $ git pull

[GitHub] storm pull request #2385: YSTORM-2727: Generic Resource Aware Scheduling

Github user govind-menon closed the pull request at: https://github.com/apache/storm/pull/2385 ---

[GitHub] storm pull request #2385: YSTORM-2727: Generic Resource Aware Scheduling

GitHub user govind-menon opened a pull request: https://github.com/apache/storm/pull/2385 YSTORM-2727: Generic Resource Aware Scheduling Remaining 1. Add Tests 2. Do more manual testing You can merge this pull request into a Git repository by running: $ git pull

[GitHub] storm pull request #2341: [STORM-2754] Not killing on exceptions in other th...

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2341#discussion_r144578588 --- Diff: storm-client/src/jvm/org/apache/storm/messaging/netty/NettyUncaughtExceptionHandler.java --- @@ -33,10 +34,9 @@ public void

[GitHub] storm pull request #2338: [STORM-2744] add in restart timeout for backpressu...

Github user govind-menon commented on a diff in the pull request: https://github.com/apache/storm/pull/2338#discussion_r140337615 --- Diff: storm-client/src/jvm/org/apache/storm/daemon/worker/Worker.java --- @@ -406,25 +406,36 @@ private WorkerBackpressureCallback

  1   2   >