Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/11644#discussion_r55908574
--- Diff: core/src/main/scala/org/apache/spark/util/Utils.scala ---
@@ -2014,8 +2014,10 @@ private[spark] object Utils extends Logging {
} catch {
case e: Exception if isBindCollision(e) =>
if (offset >= maxRetries) {
- val exceptionMessage =
- s"${e.getMessage}: Service$serviceString failed after
$maxRetries retries!"
+ val exceptionMessage = s"${e.getMessage}:
Service$serviceString failed after " +
+ s"$maxRetries retries! Consider explicitly setting the
appropriate port for" +
--- End diff --
Man, that's tricky.
But in the case where `serviceString` is empty, this would look odd:
Consider explicitly setting the appropriate port for (for example
spark.ui.port for SparkUI)
Perhaps `serviceString` should be changed so that it's "Service" if
`serviceName` is empty. Or make sure that it's never empty.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
