Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20589#discussion_r168100501
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -198,7 +200,10 @@ private[ui] class BlockDataSource(
rddPartition.storageLevel,
rddPartition.memoryUsed,
rddPartition.diskUsed,
- rddPartition.executors.mkString(" "))
+ rddPartition.executors
+ .sorted
+ .map { id => executorIdToAddress.get(id).getOrElse(id) }
--- End diff --
nit: should we sort by address?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
