Only mark tasks as forwarded if they are in the lineage cache (#3958)

This commit is contained in:
Stephanie Wang
2019-02-05 23:01:38 -08:00
committed by GitHub
parent b0fe5af7c8
commit 244fd473f4
+4 -3
View File
@@ -1948,10 +1948,11 @@ void NodeManager::ForwardTask(const Task &task, const ClientID &node_id,
if (!lineage_cache_.RemoveWaitingTask(task_id)) {
RAY_LOG(WARNING) << "Task " << task_id << " already removed from the lineage"
<< " cache. This is most likely due to reconstruction.";
} else {
// Mark as forwarded so that the task and its lineage is not
// re-forwarded in the future to the receiving node.
lineage_cache_.MarkTaskAsForwarded(task_id, node_id);
}
// Mark as forwarded so that the task and its lineage is not re-forwarded
// in the future to the receiving node.
lineage_cache_.MarkTaskAsForwarded(task_id, node_id);
// Notify the task dependency manager that we are no longer responsible
// for executing this task.