Github user vanzin commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20546#discussion_r168143242
  
    --- Diff: core/src/main/java/org/apache/spark/SparkExecutorInfo.java ---
    @@ -30,4 +30,8 @@
       int port();
       long cacheSize();
       int numRunningTasks();
    +  long usedOnHeapStorageMemory();
    --- End diff --
    
    1. That follows the names in the public `MemoryMetrics` class from the REST 
API.
    2. We could add that, just as we could add a whole lot of other things. At 
some point we should look at exposing the REST API types directly through 
`SparkStatusTracker` instead of having these mirror types.


---

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

Reply via email to