Mailing-List: contact mesos-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: mesos-dev@incubator.apache.org
Content-Type: multipart/alternative;
 boundary="===============2477354552691522836=="
MIME-Version: 1.0
Subject: Re: Review Request: Terminate executors that aren't needed.
From: "Brenden Matthews" <brenden@diddyinc.com>
To: "Vinod Kone" <vinodkone@gmail.com>,
 "mesos" <mesos-dev@incubator.apache.org>,
 "Brenden Matthews" <brenden@diddyinc.com>
Date: Mon, 03 Jun 2013 17:56:18 -0000
Message-ID: <20130603175618.22886.37484@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: "Brenden Matthews" <noreply@reviews.apache.org>
References: <20130514002048.23050.57745@reviews.apache.org>
In-Reply-To: <20130514002048.23050.57745@reviews.apache.org>
Reply-To: "Brenden Matthews" <brenden@diddyinc.com>

--===============2477354552691522836==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11125/
-----------------------------------------------------------

(Updated June 3, 2013, 5:56 p.m.)


Review request for mesos.


Description (updated)
-------

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.

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


Diffs (updated)
-----

  src/slave/slave.cpp e905ab3cd965002592c1f70c85ea3378ac8982b0 =


Diff: https://reviews.apache.org/r/11125/diff/


Testing
-------

Used in production at airbnb.


Thanks,

Brenden Matthews


--===============2477354552691522836==--