Updated a comment about resubscribing completed frameworks. Review: https://reviews.apache.org/r/63674
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/7955dc9d Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/7955dc9d Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/7955dc9d Branch: refs/heads/master Commit: 7955dc9dc2971055fde684c2e156b6eb18467306 Parents: ebd27f8 Author: Alexander Rukletsov <al...@apache.org> Authored: Wed Nov 8 11:41:19 2017 +0100 Committer: Alexander Rukletsov <al...@apache.org> Committed: Mon Nov 13 21:44:01 2017 +0100 ---------------------------------------------------------------------- src/master/master.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/mesos/blob/7955dc9d/src/master/master.cpp ---------------------------------------------------------------------- diff --git a/src/master/master.cpp b/src/master/master.cpp index 49dbaa9..59a5339 100644 --- a/src/master/master.cpp +++ b/src/master/master.cpp @@ -2667,9 +2667,8 @@ void Master::subscribe( if (validationError.isNone() && frameworkInfo.has_id() && isCompletedFramework(frameworkInfo.id())) { - // This could happen if a framework tries to subscribe after its - // failover timeout has elapsed or it unregistered itself by - // calling 'stop()' on the scheduler driver. + // This could happen if a framework tries to subscribe after its failover + // timeout has elapsed, or it has been torn down via the opeartor API. // // TODO(vinod): Master should persist admitted frameworks to the // registry and remove them from it after failover timeout.
