Re: [GitHub] storm issue #1136: [STORM-1565] Multi-Lang Performance Improvements

2018-09-26 Thread P. Taylor Goetz
Andrew, Thanks for bringing the private conversation back to the dev@ list so everyone has more context. That being said, I’d encourage the use of the public mailing lists as much as possible. Private conversations exclude the larger community from participating in potentially fruitful discuss

[GitHub] storm issue #1136: [STORM-1565] Multi-Lang Performance Improvements

2018-09-26 Thread amontalenti
Github user amontalenti commented on the issue: https://github.com/apache/storm/pull/1136 Was chatting with @roshannaik and @dan-blanchard today, and this PR came up. Someone on Storm team may benefit from taking a look at this as part of the performance revisions being done f

[GitHub] storm pull request #2851: STORM-3236 mark shutdown meters before stopping me...

2018-09-26 Thread agresch
GitHub user agresch opened a pull request: https://github.com/apache/storm/pull/2851 STORM-3236 mark shutdown meters before stopping metric reporting You can merge this pull request into a Git repository by running: $ git pull https://github.com/agresch/storm agresch_storm-323

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

2018-09-26 Thread Kishorkumar Patil
I have patch up for WorkerToken Auto renewal fix. I think this should be added to 2.x RC. https://issues.apache.org/jira/browse/STORM-3235 https://github.com/apache/storm/pull/2850 -Kishor On Tue, Sep 25, 2018 at 8:04 PM P. Taylor Goetz wrote: > Jungtaek, can you elaborate on the issues you r

[GitHub] storm pull request #2850: [STORM-3235] Fix WorkerToken renewal criteria and ...

2018-09-26 Thread kishorvpatil
GitHub user kishorvpatil opened a pull request: https://github.com/apache/storm/pull/2850 [STORM-3235] Fix WorkerToken renewal criteria and refactor - Fix broken condition to validate if new `WorkerToken` should be added. - Refactor code out of `Nimbus` into `WorkerTokenManager`