Github user xujyan commented on a diff in the pull request: https://github.com/apache/mesos/pull/279#discussion_r179547033 --- Diff: src/master/master.cpp --- @@ -6843,6 +6846,17 @@ void Master::__reregisterSlave( recoveredTasks.push_back(std::move(task)); } + foreachkey(FrameworkID frameworkId, frameworkToTasks) { --- End diff -- Can we just reference entries from `slaves.unreachableTasks` directly? Also if some items in `framework-> unreachableTasks` are already removed during recovering tasks above, they'll just be noops here.
---