[GitHub] storm issue #2613: .NET Core 2.0 multi-lang adapter

2018-04-04 Thread oizu
Github user oizu commented on the issue: https://github.com/apache/storm/pull/2613 The adapter will be published in another GitHub repository. ---

[GitHub] storm pull request #2613: .NET Core 2.0 multi-lang adapter

2018-04-04 Thread oizu
Github user oizu closed the pull request at: https://github.com/apache/storm/pull/2613 ---

[GitHub] storm issue #2618: [STORM-2905] Fix KeyNotFoundException when kill a storm a...

2018-04-04 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2618 @danny0405 I just created #2622 to fix the race condition in AsyncLocalizer. It does conflict a lot with this patch, so I wanted to make sure you saw it and had a chance to give feedback o

[GitHub] storm pull request #2622: STORM-3020: fix possible race condition in AsyncLo...

2018-04-04 Thread revans2
GitHub user revans2 opened a pull request: https://github.com/apache/storm/pull/2622 STORM-3020: fix possible race condition in AsyncLocalizer There were a number of places in AsyncLocalizer that were using synchronized to try and protect some maps. When we added in support for re

[GitHub] storm issue #2618: [STORM-2905] Fix KeyNotFoundException when kill a storm a...

2018-04-04 Thread danny0405
Github user danny0405 commented on the issue: https://github.com/apache/storm/pull/2618 @revans2 To me the race condition has none of the business of AsyncLocalize# `requestDownloadBaseTopologyBlobs `, it's the race condition on AsyncLocalize#`topologyBlobs` of timer task Asy