You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by bm...@apache.org on 2016/02/05 06:18:43 UTC

[3/8] mesos git commit: Fixed endpoint help leak upon process termination.

Fixed endpoint help leak upon process termination.

Previously, the global help process would retain the help information
of processes even after they were termianted.

This commit adds code to remove these help strings.

Review: https://reviews.apache.org/r/43215/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/0f53c9a0
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/0f53c9a0
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/0f53c9a0

Branch: refs/heads/master
Commit: 0f53c9a0bda98406b077d22f65c57f2877747754
Parents: 2eba6b1
Author: Kevin Klues <kl...@gmail.com>
Authored: Thu Feb 4 19:22:41 2016 -0800
Committer: Benjamin Mahler <be...@gmail.com>
Committed: Thu Feb 4 21:18:17 2016 -0800

----------------------------------------------------------------------
 3rdparty/libprocess/src/process.cpp | 3 +++
 1 file changed, 3 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/0f53c9a0/3rdparty/libprocess/src/process.cpp
----------------------------------------------------------------------
diff --git a/3rdparty/libprocess/src/process.cpp b/3rdparty/libprocess/src/process.cpp
index 1672e08..4a53020 100644
--- a/3rdparty/libprocess/src/process.cpp
+++ b/3rdparty/libprocess/src/process.cpp
@@ -2628,6 +2628,9 @@ void ProcessManager::cleanup(ProcessBase* process)
     delete event;
   }
 
+  // Remove help strings for all installed routes for this process.
+  dispatch(help, &Help::remove, process->pid.id);
+
   // Possible gate non-libprocess threads are waiting at.
   Gate* gate = NULL;