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: 
AppStatusStore) extends Web
             rddStorageInfo.partitions.get,
             blockPageSize,
             blockSortColumn,
    -        blockSortDesc)
    +        blockSortDesc,
    +        store.executorList(false))
    --- End diff --
    
    Should this only list active executors?
    
    Theoretically you shouldn't have any blocks stored on dead executors.


---

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

Reply via email to