[GitHub] [spark] dongjoon-hyun commented on a diff in pull request #38843: [SPARK-41327][CORE] Fix `SparkStatusTracker.getExecutorInfos` by switch On/OffHeapStorageMemory info

[GitBox <gi...@apache.org>]

dongjoon-hyun commented on code in PR #38843:
URL: https://github.com/apache/spark/pull/38843#discussion_r1035372076


##########
core/src/main/scala/org/apache/spark/SparkStatusTracker.scala:
##########
@@ -100,7 +100,7 @@ class SparkStatusTracker private[spark] (sc: SparkContext, store: AppStatusStore
   /**
    * Returns information of all known executors, including host, port, cacheSize, numRunningTasks
    * and memory metrics.
-   * Note this include information for both the driver and executors.
+   * Note this include information for both the driver and executors. 

Review Comment:
   Could you revert this change?



-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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