Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12318
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is ena
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12318#issuecomment-209095251
Thanks - merging in master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12318#issuecomment-209084165
**[Test build #2779 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2779/consoleFull)**
for PR 12318 at commit
[`0232a7a`](https://
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12318#issuecomment-209029211
**[Test build #2779 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2779/consoleFull)**
for PR 12318 at commit
[`0232a7a`](https://g
Github user chtyim commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59338307
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +156,7 @@ private[spark] class HttpServer(
throw new ServerStateExcept
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59335396
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +156,7 @@ private[spark] class HttpServer(
throw new ServerStateExcept
Github user chtyim commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59331847
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +156,7 @@ private[spark] class HttpServer(
throw new ServerStateExcept
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59328586
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +156,7 @@ private[spark] class HttpServer(
throw new ServerStateExceptio
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59328490
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +156,7 @@ private[spark] class HttpServer(
throw new ServerStateExceptio
Github user chtyim commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59326211
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +156,7 @@ private[spark] class HttpServer(
throw new ServerStateExcept
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59325455
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +156,7 @@ private[spark] class HttpServer(
throw new ServerStateExceptio
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59325045
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +156,7 @@ private[spark] class HttpServer(
throw new ServerStateExceptio
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59324981
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +158,7 @@ private[spark] class HttpServer(
throw new ServerStateExceptio
Github user chtyim commented on the pull request:
https://github.com/apache/spark/pull/12318#issuecomment-208718393
Addressed comment. Please have a look again. Thanks.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user chtyim commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59320494
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +158,7 @@ private[spark] class HttpServer(
throw new ServerStateExcept
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59318036
--- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala ---
@@ -350,4 +352,10 @@ private[spark] object JettyUtils extends Logging {
private[spark
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59318015
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +158,7 @@ private[spark] class HttpServer(
throw new ServerStateExceptio
Github user chtyim commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59317619
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +158,7 @@ private[spark] class HttpServer(
throw new ServerStateExcept
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/12318#discussion_r59314056
--- Diff: core/src/main/scala/org/apache/spark/HttpServer.scala ---
@@ -155,6 +158,7 @@ private[spark] class HttpServer(
throw new ServerStateExcept
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12318#issuecomment-208671355
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your p
GitHub user chtyim opened a pull request:
https://github.com/apache/spark/pull/12318
[SPARK-14513][CORE] Fix threads left behind after stopping SparkContext
## What changes were proposed in this pull request?
Shutting down `QueuedThreadPool` used by Jetty `Server` to avoid t
21 matches
Mail list logo