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();