[GitHub] [spark] Ngone51 commented on a change in pull request #28880: [SPARK-29465][YARN][WEBUI] Adding Check to not to set UI port (spark.ui.port) property if mentioned explicitly

Ngone51 commented on a change in pull request #28880: URL: https://github.com/apache/spark/pull/28880#discussion_r444658287 ## File path: resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala ## @@ -211,9 +211,11 @@ private[spark] class

[GitHub] [spark] Ngone51 commented on a change in pull request #28880: [SPARK-29465][YARN][WEBUI] Adding Check to not to set UI port (spark.ui.port) property if mentioned explicitly

Ngone51 commented on a change in pull request #28880: URL: https://github.com/apache/spark/pull/28880#discussion_r444621048 ## File path: resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala ## @@ -211,9 +211,11 @@ private[spark] class

[GitHub] [spark] Ngone51 commented on a change in pull request #28880: [SPARK-29465][YARN][WEBUI] Adding Check to not to set UI port (spark.ui.port) property if mentioned explicitly

Ngone51 commented on a change in pull request #28880: URL: https://github.com/apache/spark/pull/28880#discussion_r443933600 ## File path: resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala ## @@ -211,9 +211,11 @@ private[spark] class

[GitHub] [spark] Ngone51 commented on a change in pull request #28880: [SPARK-29465][YARN][WEBUI] Adding Check to not to set UI port (spark.ui.port) property if mentioned explicitly

Ngone51 commented on a change in pull request #28880: URL: https://github.com/apache/spark/pull/28880#discussion_r443287907 ## File path: resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala ## @@ -211,9 +211,11 @@ private[spark] class