From 8fa00dd05f3b5e78e9268715b0f05f9f739eca5c Mon Sep 17 00:00:00 2001
From: bomeng <bmeng@us.ibm.com>
Date: Fri, 3 Jun 2016 09:59:15 -0700
Subject: [PATCH] [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 <bmeng@us.ibm.com>
Closes #13475 from bomeng/SPARK-15737.
---
.../org/apache/spark/deploy/rest/RestSubmissionServer.scala | 1 +
core/src/main/scala/org/apache/spark/ui/JettyUtils.scala | 1 +
2 files changed, 2 insertions(+)
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 7e93bfc45e..fa55d47084 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)
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 6854f7baf7..50283f2b74 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 {
--
GitLab