-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65109/
-----------------------------------------------------------
(Updated Feb. 6, 2018, 9:45 a.m.)
Review request for mesos, Benjamin Mahler and Vinod Kone.
Changes
-------
Patch summary updated to refect that it is the first half of a fix.
Summary (updated)
-----------------
Fixed a bug relating to lingering executors [1/2].
Bugs: MESOS-8411
https://issues.apache.org/jira/browse/MESOS-8411
Repository: mesos
Description
-------
An executor should be shutdown if all of its tasks are
killed while the executor is launching.
This patch fixes and issue where the executor is left
running when the task(s) get killed between the executor
registration/subscription and `Slave::___run()`. See
MESOS-8411 for more details. There is an additional race
in the agent failover case that is addressed in this patch.
The fix here is to fix the race by checking an executor's various
tasks queues during task kill and executor (re-)registration,
and shutting down executors that had never received any tasks.
Diffs (updated)
-----
src/slave/constants.hpp e6cb7cc0ccdaaf981eb66defa21b38720f4e1de9
src/slave/slave.hpp 09c01ebd1b5e8008ba9e7d412042f1db76a2c5a5
src/slave/slave.cpp a6a5c93ab2d541c870cb52587495de20ed5ac1f4
Diff: https://reviews.apache.org/r/65109/diff/7/
Changes: https://reviews.apache.org/r/65109/diff/6-7/
Testing
-------
make check
new tests in #65111
Thanks,
Meng Zhu
