Skip to content

Commit c0e7e81

Browse files
committed
Merge pull request #465 from depay/master
Remove persistence task after mesos task running
2 parents 2f9b7d3 + 495bdf2 commit c0e7e81

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/main/scala/org/apache/mesos/chronos/scheduler/mesos/MesosJobFramework.scala

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -175,9 +175,6 @@ import scala.concurrent.ExecutionContext.Implicits.global
175175
)
176176
if (status == Protos.Status.DRIVER_RUNNING) {
177177
for (task <- tasks) {
178-
val deleted = taskManager.removeTask(task._1)
179-
log.fine("Successfully launched task '%s' via chronos, task records successfully deleted: '%b'"
180-
.format(task._1, deleted))
181178
runningTasks.put(task._2.name, new ChronosTask(task._3.getSlaveId.getValue))
182179

183180
//TODO(FL): Handle case if chronos can't launch the task.
@@ -228,7 +225,8 @@ import scala.concurrent.ExecutionContext.Implicits.global
228225
log.info("Task with id '%s' LOST".format(taskStatus.getTaskId.getValue))
229226
scheduler.handleFailedTask(taskStatus)
230227
case TaskState.TASK_RUNNING =>
231-
log.info("Task with id '%s' RUNNING.".format(taskStatus.getTaskId.getValue))
228+
log.info("Task with id '%s' RUNNING. Removing persistence task.".format(taskStatus.getTaskId.getValue))
229+
taskManager.removeTask(taskStatus.getTaskId.getValue)
232230
case TaskState.TASK_KILLED =>
233231
log.info("Task with id '%s' KILLED.".format(taskStatus.getTaskId.getValue))
234232
scheduler.handleKilledTask(taskStatus)

0 commit comments

Comments
 (0)