You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by va...@apache.org on 2016/06/03 16:59:21 UTC
spark git commit: [SPARK-15737][CORE] fix jetty warning
Repository: spark
Updated Branches:
refs/heads/master c2f0cb4f6 -> 8fa00dd05
[SPARK-15737][CORE] fix jetty warning
## What changes were proposed in this pull request?
After upgrading Jetty to 9.2, we always see "WARN org.eclipse.jetty.server.handler.AbstractHandler: No Server set for org.eclipse.jetty.server.handler.ErrorHandler" while running any test cases.
This PR will fix it.
## How was this patch tested?
The existing test cases will cover it.
Author: bomeng <bm...@us.ibm.com>
Closes #13475 from bomeng/SPARK-15737.
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/8fa00dd0
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/8fa00dd0
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/8fa00dd0
Branch: refs/heads/master
Commit: 8fa00dd05f3b5e78e9268715b0f05f9f739eca5c
Parents: c2f0cb4
Author: bomeng <bm...@us.ibm.com>
Authored: Fri Jun 3 09:59:15 2016 -0700
Committer: Marcelo Vanzin <va...@cloudera.com>
Committed: Fri Jun 3 09:59:15 2016 -0700
----------------------------------------------------------------------
.../scala/org/apache/spark/deploy/rest/RestSubmissionServer.scala | 1 +
core/src/main/scala/org/apache/spark/ui/JettyUtils.scala | 1 +
2 files changed, 2 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/spark/blob/8fa00dd0/core/src/main/scala/org/apache/spark/deploy/rest/RestSubmissionServer.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/deploy/rest/RestSubmissionServer.scala b/core/src/main/scala/org/apache/spark/deploy/rest/RestSubmissionServer.scala
index 7e93bfc..fa55d47 100644
--- a/core/src/main/scala/org/apache/spark/deploy/rest/RestSubmissionServer.scala
+++ b/core/src/main/scala/org/apache/spark/deploy/rest/RestSubmissionServer.scala
@@ -89,6 +89,7 @@ private[spark] abstract class RestSubmissionServer(
server.addConnector(connector)
val mainHandler = new ServletContextHandler
+ mainHandler.setServer(server)
mainHandler.setContextPath("/")
contextToServlet.foreach { case (prefix, servlet) =>
mainHandler.addServlet(new ServletHolder(servlet), prefix)
http://git-wip-us.apache.org/repos/asf/spark/blob/8fa00dd0/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala b/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
index 6854f7b..50283f2 100644
--- a/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
+++ b/core/src/main/scala/org/apache/spark/ui/JettyUtils.scala
@@ -291,6 +291,7 @@ private[spark] object JettyUtils extends Logging {
val errorHandler = new ErrorHandler()
errorHandler.setShowStacks(true)
+ errorHandler.setServer(server)
server.addBean(errorHandler)
server.setHandler(collection)
try {
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org