Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20327#discussion_r175533480
--- Diff: core/src/main/scala/org/apache/spark/ui/WebUI.scala ---
@@ -126,7 +126,11 @@ private[spark] abstract class WebUI(
def bind(): Unit = {
assert(serverInfo.isEmpty, s"Attempted to bind $className more than
once!")
try {
- val host = Option(conf.getenv("SPARK_LOCAL_IP")).getOrElse("0.0.0.0")
+ val host = if (Utils.isClusterMode(conf)) {
--- End diff --
Sorry, but this is still bothering me. It probably works fine in YARN mode
because the UI is proxied by the RM. But that's not true for any other mode,
and the default before was to bind to `0.0.0.0` and now it is binding to a
specific hostname. And that may not be the public one in a multi-homed machine,
so now the UI wouldn't be accessible without extra configuration.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
