Repository: spark
Updated Branches:
  refs/heads/branch-1.2 1f8b71868 -> c573af4f1


[SPARK-5402] log executor ID at executor-construction time

also rename "slaveHostname" to "executorHostname"

Author: Ryan Williams <[email protected]>

Closes #4195 from ryan-williams/exec and squashes the following commits:

e60a7bb [Ryan Williams] log executor ID at executor-construction time

(cherry picked from commit aea25482c370fbcf712a464501605bc16ee4ed5d)
Signed-off-by: Andrew Or <[email protected]>

Conflicts:
        core/src/main/scala/org/apache/spark/executor/Executor.scala


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/c573af4f
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/c573af4f
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/c573af4f

Branch: refs/heads/branch-1.2
Commit: c573af4f1d3022a4712458a4b3b5061a8017e040
Parents: 1f8b718
Author: Ryan Williams <[email protected]>
Authored: Sun Jan 25 14:20:02 2015 -0800
Committer: Andrew Or <[email protected]>
Committed: Sun Jan 25 14:21:21 2015 -0800

----------------------------------------------------------------------
 .../scala/org/apache/spark/executor/Executor.scala     | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/c573af4f/core/src/main/scala/org/apache/spark/executor/Executor.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/executor/Executor.scala 
b/core/src/main/scala/org/apache/spark/executor/Executor.scala
index da030f2..eaf0c82 100644
--- a/core/src/main/scala/org/apache/spark/executor/Executor.scala
+++ b/core/src/main/scala/org/apache/spark/executor/Executor.scala
@@ -41,13 +41,16 @@ import 
org.apache.spark.util.{SparkUncaughtExceptionHandler, AkkaUtils, Utils}
  */
 private[spark] class Executor(
     executorId: String,
-    slaveHostname: String,
+    executorHostname: String,
     properties: Seq[(String, String)],
     numCores: Int,
     isLocal: Boolean = false,
     actorSystem: ActorSystem = null)
   extends Logging
 {
+
+  logInfo(s"Starting executor ID $executorId on host $executorHostname")
+
   // Application dependencies (added through SparkContext) that we've fetched 
so far on this node.
   // Each map holds the master's timestamp for the version of that file or JAR 
we got.
   private val currentFiles: HashMap[String, Long] = new HashMap[String, Long]()
@@ -58,12 +61,12 @@ private[spark] class Executor(
   @volatile private var isStopped = false
 
   // No ip or host:port - just hostname
-  Utils.checkHost(slaveHostname, "Expected executed slave to be a hostname")
+  Utils.checkHost(executorHostname, "Expected executed slave to be a hostname")
   // must not have port specified.
-  assert (0 == Utils.parseHostPort(slaveHostname)._2)
+  assert (0 == Utils.parseHostPort(executorHostname)._2)
 
   // Make sure the local hostname we report matches the cluster scheduler's 
name for this host
-  Utils.setCustomHostname(slaveHostname)
+  Utils.setCustomHostname(executorHostname)
 
   // Set spark.* properties from executor arg
   val conf = new SparkConf(true)
@@ -84,7 +87,7 @@ private[spark] class Executor(
     if (!isLocal) {
       val port = conf.getInt("spark.executor.port", 0)
       val _env = SparkEnv.createExecutorEnv(
-        conf, executorId, slaveHostname, port, numCores, isLocal, actorSystem)
+        conf, executorId, executorHostname, port, numCores, isLocal, 
actorSystem)
       SparkEnv.set(_env)
       _env.metricsSystem.registerSource(executorSource)
       _env.blockManager.initialize(conf.getAppId)


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to