akshatb1 commented on a change in pull request #28429:
URL: https://github.com/apache/spark/pull/28429#discussion_r419499799
##########
File path:
core/src/main/scala/org/apache/spark/deploy/StandaloneResourceUtils.scala
##########
@@ -142,9 +142,13 @@ private[spark] object StandaloneResourceUtils extends
Logging {
// used for UI
def formatResourcesAddresses(resources: Map[String, ResourceInformation]):
String = {
- resources.map { case (rName, rInfo) =>
- s"$rName: ${rInfo.addresses.mkString("[", ", ", "]")}"
- }.mkString(", ")
+ if (resources != null) {
Review comment:
Thanks @tgravescs. I will fix it in the DriverInfo. With that change, do
we still want to keep this change in StandaloneResourceUtils.scala? I had
presumed that the null value was as intended for the drivers which are not
launched, hence made above fix.
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
