[GitHub] storm pull request #2741: STORM-3124 reconnect to pacemaker on failure

2018-06-29 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2741#discussion_r199193610 --- Diff: storm-client/src/jvm/org/apache/storm/messaging/netty/KerberosSaslClientHandler.java --- @@ -96,53 +96,69 @@ private void

[GitHub] storm pull request #2741: STORM-3124 reconnect to pacemaker on failure

2018-06-29 Thread agresch
Github user agresch commented on a diff in the pull request: https://github.com/apache/storm/pull/2741#discussion_r199145547 --- Diff: storm-client/src/jvm/org/apache/storm/messaging/netty/KerberosSaslClientHandler.java --- @@ -96,53 +96,69 @@ private void

[GitHub] storm pull request #2741: STORM-3124 reconnect to pacemaker on failure

2018-06-28 Thread srdo
Github user srdo commented on a diff in the pull request: https://github.com/apache/storm/pull/2741#discussion_r199057391 --- Diff: storm-client/src/jvm/org/apache/storm/messaging/netty/KerberosSaslClientHandler.java --- @@ -96,53 +96,69 @@ private void

[GitHub] storm pull request #2741: STORM-3124 reconnect to pacemaker on failure

2018-06-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/storm/pull/2741 ---

[GitHub] storm pull request #2741: STORM-3124 reconnect to pacemaker on failure

2018-06-28 Thread danny0405
Github user danny0405 commented on a diff in the pull request: https://github.com/apache/storm/pull/2741#discussion_r198736753 --- Diff: storm-client/src/jvm/org/apache/storm/messaging/netty/KerberosSaslClientHandler.java --- @@ -96,53 +96,69 @@ private void

[GitHub] storm pull request #2741: STORM-3124 reconnect to pacemaker on failure

2018-06-27 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2741#discussion_r198638777 --- Diff: storm-client/src/jvm/org/apache/storm/pacemaker/PacemakerClient.java --- @@ -153,13 +154,13 @@ public String secretKey() { }

[GitHub] storm pull request #2741: STORM-3124 reconnect to pacemaker on failure

2018-06-27 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/2741#discussion_r198638417 --- Diff: storm-client/src/jvm/org/apache/storm/messaging/netty/KerberosSaslClientHandler.java --- @@ -96,53 +96,69 @@ private void

[GitHub] storm pull request #2741: STORM-3124 reconnect to pacemaker on failure

2018-06-27 Thread agresch
GitHub user agresch opened a pull request: https://github.com/apache/storm/pull/2741 STORM-3124 reconnect to pacemaker on failure You can merge this pull request into a Git repository by running: $ git pull https://github.com/agresch/storm agresch_pacemaker_connect