switched to string interpolation

markhamstra · Sep 8, 2014 · bdd0928 · bdd0928
1 parent 1dd591b
commit bdd0928
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/core/src/main/scala/org/apache/spark/deploy/master/Master.scala b/core/src/main/scala/org/apache/spark/deploy/master/Master.scala
@@ -302,7 +302,7 @@ private[spark] class Master(
           exec.application.driver ! ExecutorUpdated(execId, state, message, exitStatus)
           if (ExecutorState.isFinished(state)) {
             // Remove this executor from the worker and app
-            logInfo("Removing executor " + exec.fullId + " because it is " + state)
+            logInfo(s"Removing executor ${exec.fullId} because it is $state")
             appInfo.removeExecutor(exec)
             exec.worker.removeExecutor(exec)
 
@@ -312,8 +312,8 @@ private[spark] class Master(
               if (appInfo.incrementRetryCount() < ApplicationState.MAX_NUM_RETRY) {
                 schedule()
               } else {
-                logError("Application %s with ID %s failed %d times, removing it".format(
-                  appInfo.desc.name, appInfo.id, appInfo.retryCount))
+                logError(s"Application ${appInfo.desc.name} with ID ${appInfo.id} failed " +
+                  s"${appInfo.retryCount} times; removing it")
                 val execs = idToApp(appId).executors.values
                 if (!execs.exists(_.state == ExecutorState.RUNNING)) {
                   removeApplication(appInfo, ApplicationState.FAILED)
@@ -323,7 +323,7 @@ private[spark] class Master(
           }
         }
         case None =>
-          logWarning("Got status update for unknown executor " + appId + "/" + execId)
+          logWarning(s"Got status update for unknown executor $appId/$execId")
       }
     }