[GitHub] spark issue #19217: [SPARK-21991][LAUNCHER] Fix race condition in LauncherSe...

2017-10-24 Thread nivox
Github user nivox commented on the issue: https://github.com/apache/spark/pull/19217 @vanzin @ash211 I just modified the title of the PR as per your suggestion --- - To unsubscribe, e-mail: reviews-unsubscr

[GitHub] spark pull request #19217: [SPARK-21991][LAUNCHER] LauncherServer acceptConn...

2017-10-24 Thread nivox
Github user nivox commented on a diff in the pull request: https://github.com/apache/spark/pull/19217#discussion_r146519805 --- Diff: launcher/src/main/java/org/apache/spark/launcher/LauncherServer.java --- @@ -232,20 +232,20 @@ public void run

[GitHub] spark pull request #19217: [SPARK-21991][LAUNCHER] LauncherServer acceptConn...

2017-10-24 Thread nivox
Github user nivox commented on a diff in the pull request: https://github.com/apache/spark/pull/19217#discussion_r146518809 --- Diff: launcher/src/main/java/org/apache/spark/launcher/LauncherServer.java --- @@ -232,20 +232,20 @@ public void run

[GitHub] spark pull request #19217: [SPARK-21991][LAUNCHER] LauncherServer acceptConn...

2017-09-13 Thread nivox
GitHub user nivox opened a pull request: https://github.com/apache/spark/pull/19217 [SPARK-21991][LAUNCHER] LauncherServer acceptConnections thread sometime dies if machine has very high load ## What changes were proposed in this pull request? This patch changes the order in