[GitHub] spark pull request #20867: [SPARK-23759][UI] Unable to bind Spark2 history s...

2018-03-22 Thread felixalbani
Github user felixalbani closed the pull request at:

https://github.com/apache/spark/pull/20867


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request #20867: [SPARK-23759][UI] Unable to bind Spark2 history s...

2018-03-21 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request:

https://github.com/apache/spark/pull/20867#discussion_r176068898
  
--- Diff: core/src/main/scala/org/apache/spark/ui/JettyUtils.scala ---
@@ -330,12 +330,13 @@ private[spark] object JettyUtils extends Logging {
   -1,
   connectionFactories: _*)
 connector.setPort(port)
+connector.setHost(hostName)
 connector.start()
 
 // Currently we only use "SelectChannelConnector"
 // Limit the max acceptor number to 8 so that we don't waste a lot 
of threads
 connector.setAcceptQueueSize(math.min(connector.getAcceptors, 8))
--- End diff --

I think also this line should be moved before the start.


---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org