[GitHub] storm pull request #2892: Added in better docs for local mode testing.

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

[GitHub] storm pull request #2891: STORM-3269: Update version of httpclient, and fix ...

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

[GitHub] storm pull request #2882: STORM-3260: Add in support to print some state

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

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

2018-10-23 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/2893#discussion_r227445654 --- Diff: storm-core/src/native/worker-launcher/impl/worker-launcher.c --- @@ -593,14 +593,17 @@ int recursive_delete(const char *path, int supervisor_owns_dir)

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

2018-10-23 Thread agresch
Github user agresch commented on a diff in the pull request: https://github.com/apache/storm/pull/2893#discussion_r227434073 --- Diff: storm-core/src/native/worker-launcher/impl/worker-launcher.c --- @@ -593,14 +593,17 @@ int recursive_delete(const char *path, int

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

2018-10-23 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2893#discussion_r227418284 --- Diff: storm-core/src/native/worker-launcher/impl/worker-launcher.c --- @@ -593,14 +593,17 @@ int recursive_delete(const char *path, int

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

2018-10-23 Thread govind-menon
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 #2893: STORM-3272 allow worker-launcher to delete dead sy...

2018-10-23 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/storm/pull/2893#discussion_r227412685 --- Diff: storm-core/src/native/worker-launcher/impl/worker-launcher.c --- @@ -593,14 +593,17 @@ int recursive_delete(const char *path, int supervisor_owns_dir)

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

2018-10-23 Thread revans2
GitHub user revans2 opened a pull request: https://github.com/apache/storm/pull/2894 STORM-3273: Remove storm.local.hostname from topology conf. You can merge this pull request into a Git repository by running: $ git pull https://github.com/revans2/incubator-storm STORM-3273

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

2018-10-23 Thread agresch
GitHub user agresch opened a pull request: https://github.com/apache/storm/pull/2893 STORM-3272 allow worker-launcher to delete dead symlinks When a topology folder contains a symlink to a missing file, the worker-launcher would not remove the symlink. This prevents force delete

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

2018-10-23 Thread Julien Nioche
Great, thanks Bobby On Mon, 22 Oct 2018 at 18:31, Bobby Evans wrote: > Julien, > > I have put up pull requests for the docs and for fixing some of the issues > with LocalCluster that you found. > > https://github.com/apache/storm/pull/2891 > > https://github.com/apache/storm/pull/2892 > > The