You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by vi...@apache.org on 2016/03/16 01:03:38 UTC
[3/5] mesos git commit: Close the connection upon framework teardown
for HTTP frameworks.
Close the connection upon framework teardown for HTTP frameworks.
This change fixes a `TODO` and closes the connection upon receieving a
teardown request from the HTTP framework.
Review: https://reviews.apache.org/r/44729/
Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/92b35417
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/92b35417
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/92b35417
Branch: refs/heads/master
Commit: 92b35417b8b44f1eeaa19638c5d4bc6b9c52a0bd
Parents: b862334
Author: Anand Mazumdar <ma...@gmail.com>
Authored: Tue Mar 15 17:03:05 2016 -0700
Committer: Vinod Kone <vi...@gmail.com>
Committed: Tue Mar 15 17:03:05 2016 -0700
----------------------------------------------------------------------
src/master/master.cpp | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/mesos/blob/92b35417/src/master/master.cpp
----------------------------------------------------------------------
diff --git a/src/master/master.cpp b/src/master/master.cpp
index d0380db..e6290ea 100644
--- a/src/master/master.cpp
+++ b/src/master/master.cpp
@@ -6166,7 +6166,10 @@ void Master::removeFramework(Framework* framework)
// TODO(benh): unlink(framework->pid);
- // TODO(anand): For http frameworks, close the connection.
+ // For http frameworks, close the connection.
+ if (framework->http.isSome()) {
+ framework->http->close();
+ }
framework->unregisteredTime = Clock::now();