[SPARK-23888][CORE] correct the comment of hasAttemptOnHost()

Ngone51 · squito · commit c8f3ac69d176 · 2018-04-23T15:35:50.000-05:00
TaskSetManager.hasAttemptOnHost had a misleading comment. The comment said that it only checked for running tasks, but really it checked for any tasks that might have run in the past as well. This updates to line up with the implementation. Author: wuyi <ngone_5451@163.com> Closes #20998 from Ngone51/SPARK-23888.
diff --git a/core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala b/core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala
@@ -287,7 +287,7 @@ private[spark] class TaskSetManager(
     None
   }
 
-  /** Check whether a task is currently running an attempt on a given host */
+  /** Check whether a task once ran an attempt on a given host */
   private def hasAttemptOnHost(taskIndex: Int, host: String): Boolean = {
     taskAttempts(taskIndex).exists(_.host == host)
   }

Original file line number	Diff line number	Diff line change
`@@ -287,7 +287,7 @@ private[spark] class TaskSetManager(`
`287`	`287`	`None`
`288`	`288`	`}`
`289`	`289`
`290`		`- /** Check whether a task is currently running an attempt on a given host */`
	`290`	`+ /** Check whether a task once ran an attempt on a given host */`
`291`	`291`	`private def hasAttemptOnHost(taskIndex: Int, host: String): Boolean = {`
`292`	`292`	`taskAttempts(taskIndex).exists(_.host == host)`
`293`	`293`	`}`