[GitHub] storm pull request #2843: STORM-3230: Add in sync if key not found

2018-09-20 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2843 ---

[GitHub] storm pull request #2843: STORM-3230: Add in sync if key not found

2018-09-19 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2843#discussion_r218922619 --- Diff: storm-client/src/jvm/org/apache/storm/security/auth/workertoken/WorkerTokenAuthorizer.java --- @@ -95,9 +95,12 @@ private static IStormClusterStat

[GitHub] storm pull request #2843: STORM-3230: Add in sync if key not found

2018-09-19 Thread Ethanlm
Github user Ethanlm commented on a diff in the pull request: https://github.com/apache/storm/pull/2843#discussion_r218905963 --- Diff: storm-client/src/jvm/org/apache/storm/security/auth/workertoken/WorkerTokenAuthorizer.java --- @@ -95,9 +95,12 @@ private static IStormClusterStat

[GitHub] storm pull request #2843: STORM-3230: Add in sync if key not found

2018-09-19 Thread agresch
Github user agresch commented on a diff in the pull request: https://github.com/apache/storm/pull/2843#discussion_r218902320 --- Diff: storm-client/src/jvm/org/apache/storm/cluster/StormClusterStateImpl.java --- @@ -903,6 +911,7 @@ public void removeAllPrivateWorkerKeys(String top

[GitHub] storm pull request #2843: STORM-3230: Add in sync if key not found

2018-09-19 Thread revans2
GitHub user revans2 opened a pull request: https://github.com/apache/storm/pull/2843 STORM-3230: Add in sync if key not found You can merge this pull request into a Git repository by running: $ git pull https://github.com/revans2/incubator-storm STORM-3230 Alternatively you c