tiehexue commented on a change in pull request #32326:
URL: https://github.com/apache/spark/pull/32326#discussion_r619775081
##########
File path: core/src/main/scala/org/apache/spark/SparkContext.scala
##########
@@ -1812,6 +1812,13 @@ class SparkContext(config: SparkConf) extends Logging {
/** The version of Spark on which this application is running. */
def version: String = SPARK_VERSION
+ /**
+ * Return an array of executors' host name from the block manager.
+ */
+ def getExecutorHosts: Array[String] = {
Review comment:
Thanks for your comment. Your comment just remind me there is a
typo-error in previous commit. It should be:
getExecutorMemoryStatus.map(_._1.split(":")(0)).toArray
It is not an api in my first thought. But the splitting here is kind of
messy in other place. Making it as an api would help me out, and would enclose
implementation of how we get the host name list.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
