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

[GitBox <gi...@apache.org>]

rajatahujaatinmobi commented on pull request #28880:
URL: https://github.com/apache/spark/pull/28880#issuecomment-647203127


   > @rajatahujaatinmobi . This is a partial revert of [SPARK-3627](https://issues.apache.org/jira/browse/SPARK-3627) and has been discussed before. Please see [SPARK-29465](https://issues.apache.org/jira/browse/SPARK-29465).
   > The last advice was `port range` might be acceptable solution.
   
   
   If spark.ui.port is Set then it will start from that port and will try web ui until spark.port.maxRetries and its default value is 16. So it takes care of range. 
   ```
   
   spark.port.maxRetries | 16 | Maximum number of retries when binding to a port before giving up.     When a port is given a specific value (non 0), each subsequent retry will     increment the port used in the previous attempt by 1 before retrying. This     essentially allows it to try a range of ports from the start port specified     to port + maxRetries.
   -- | -- | --
   
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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