Skip to content
Snippets Groups Projects
Commit b579b832 authored by Matei Zaharia's avatar Matei Zaharia
Browse files

Merge pull request #306 from kayousterhout/remove_pending

Remove unused hasPendingTasks methods
parents 19672dca 8419148e
No related branches found
No related tags found
No related merge requests found
......@@ -117,8 +117,4 @@ private[spark] class Pool(
parent.decreaseRunningTasks(taskNum)
}
}
override def hasPendingTasks(): Boolean = {
schedulableQueue.exists(_.hasPendingTasks())
}
}
......@@ -42,5 +42,4 @@ private[spark] trait Schedulable {
def executorLost(executorId: String, host: String): Unit
def checkSpeculatableTasks(): Boolean
def getSortedTaskSetQueue(): ArrayBuffer[TaskSetManager]
def hasPendingTasks(): Boolean
}
......@@ -365,13 +365,6 @@ private[spark] class TaskSchedulerImpl(
}
}
// Check for pending tasks in all our active jobs.
def hasPendingTasks: Boolean = {
synchronized {
rootPool.hasPendingTasks()
}
}
def executorLost(executorId: String, reason: ExecutorLossReason) {
var failedExecutor: Option[String] = None
......
......@@ -681,10 +681,6 @@ private[spark] class TaskSetManager(
return foundTasks
}
override def hasPendingTasks(): Boolean = {
numTasks > 0 && tasksSuccessful < numTasks
}
private def getLocalityWait(level: TaskLocality.TaskLocality): Long = {
val defaultWait = System.getProperty("spark.locality.wait", "3000")
level match {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment