[GitHub] storm issue #2723: [STORM-3112] Incremental scheduling supports

2018-10-24 Thread danny0405
Github user danny0405 commented on the issue: https://github.com/apache/storm/pull/2723 @revans2 Hi, Bobby, do you have time to review this patch now, cause it's a little long time and i kind of forget details about this patch, thx in advance. ---

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

2018-10-24 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r227979286 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -119,15 +120,81 @@ // tests by subclassing. private static Utils

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

2018-10-24 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r227972886 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -119,15 +120,81 @@ // tests by subclassing. private static Utils

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

2018-10-24 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r227936615 --- Diff: storm-client/src/jvm/org/apache/storm/daemon/worker/WorkerState.java --- @@ -155,6 +156,7 @@ public WorkerState(Map conf, IContext mqContext, String

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

2018-10-24 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r227896771 --- Diff: storm-client/src/jvm/org/apache/storm/Config.java --- @@ -1041,6 +1041,19 @@ @isPositiveNumber @NotNull public static final

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

2018-10-24 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r227979722 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -119,15 +120,81 @@ // tests by subclassing. private static Utils

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

2018-10-24 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r227976069 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -119,15 +120,81 @@ // tests by subclassing. private static Utils

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

2018-10-24 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r227975023 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -121,11 +121,68 @@ private static String memoizedLocalHostnameString = null;

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

2018-10-24 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/2881#discussion_r227974287 --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java --- @@ -119,15 +120,81 @@ // tests by subclassing. private static Utils

[GitHub] storm pull request #2895: [STORM-3275] Fix UIHelpers timeout while starting ...

2018-10-24 Thread kishorvpatil
GitHub user kishorvpatil opened a pull request: https://github.com/apache/storm/pull/2895 [STORM-3275] Fix UIHelpers timeout while starting profiler The Jprofiler start action, requires calculating timeout in milliseconds, while input is in minutes. Also, the logic works to say,

[CANCELED] [VOTE] Release Apache Storm 2.0.0 (rc3)

2018-10-24 Thread P. Taylor Goetz
Canceling this vote due to blocker issues. -Taylor > On Oct 16, 2018, at 3:48 PM, P. Taylor Goetz wrote: > > This is a call to vote on releasing Apache Storm 2.0.0 (rc3) > > Full list of changes in this release: > >

[GitHub] storm pull request #2894: STORM-3273: Remove storm.local.hostname from topol...

2018-10-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2894 ---

[GitHub] storm pull request #2893: STORM-3272 allow worker-launcher to delete dead sy...

2018-10-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2893 ---