Skip to content

Commit

Permalink
[SPARK-23888][CORE] correct the comment of hasAttemptOnHost()
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
Ngone51 authored and squito committed Apr 23, 2018
1 parent e82cb68 commit c8f3ac6
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -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)
}
Expand Down

0 comments on commit c8f3ac6

Please sign in to comment.