----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10932/ -----------------------------------------------------------
(Updated May 6, 2013, 7:03 p.m.) Review request for mesos. Changes ------- @ Ben: Is this what you had in mind? Description ------- >From 607072595b91993e2d47251ee841fb3dc5d84e05 Mon Sep 17 00:00:00 2001 From: Brenden Matthews <[email protected]> Date: Fri, 3 May 2013 09:47:22 -0700 Subject: [PATCH 8/9] Terminate executors that aren't needed. If we launch an executor and then kill the task immediately after, make sure we also terminate the executor when there are no other tasks. --- src/slave/slave.cpp | 48 +++++++++++++++++++++++++++--------------------- 1 file changed, 27 insertions(+), 21 deletions(-) Diffs (updated) ----- include/mesos/executor.hpp 9b25834 src/exec/exec.cpp 1f022ca Diff: https://reviews.apache.org/r/10932/diff/ Testing ------- Used in production at airbnb. Thanks, Brenden Matthews
