Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20589
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/20589#discussion_r168141926
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -198,7 +200,10 @@ private[ui] class BlockDataSource(
rddPartiti
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(
rddPartiti
Github user attilapiros commented on a diff in the pull request:
https://github.com/apache/spark/pull/20589#discussion_r167884170
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -76,7 +76,8 @@ private[ui] class RDDPage(parent: SparkUITab, store:
AppSt
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20589#discussion_r167880936
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -76,7 +76,8 @@ private[ui] class RDDPage(parent: SparkUITab, store:
AppStatusS
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20589#discussion_r167666745
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -198,7 +200,7 @@ private[ui] class BlockDataSource(
rddPartition.sto
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20589#discussion_r167666944
--- Diff: core/src/main/scala/org/apache/spark/ui/storage/RDDPage.scala ---
@@ -243,7 +246,8 @@ private[ui] class BlockPagedTable(
rddPartitions,
GitHub user attilapiros opened a pull request:
https://github.com/apache/spark/pull/20589
[SPARK-23394][UI] In RDD storage page show the executor addresses instead
of the IDs
## What changes were proposed in this pull request?
Extending RDD storage page to show executor add
