-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65449/
-----------------------------------------------------------
(Updated Feb. 12, 2018, 4:11 p.m.)
Review request for mesos, Benjamin Mahler, Chun-Hung Hsiao, and Vinod Kone.
Changes
-------
`"Executor" + executor->state` was complained on Mac, changed it to plain
string.
Bugs: MESOS-1720
https://issues.apache.org/jira/browse/MESOS-1720
Repository: mesos
Description
-------
Master relies on `ExitedExecutorMessage` from the agent to recycle
executor entry. However, this message won't be sent if the executor
never actually launched (due to transient error), leaving executor
info on the master lingering and resource claimed.
See MESOS-1720.
This patch fixes this issue by sending the `ExitedExecutorMessage`
from the agent if the executor is never launched.
Diffs (updated)
-----
src/slave/slave.hpp 30151c4886e12e9183a971b86b854e28a8ca1b39
src/slave/slave.cpp f98f37321872d090176b7cc50873fc3c627773f5
src/tests/mock_slave.hpp 942ead57fc67bdd2a268c67575952349838dc280
src/tests/mock_slave.cpp 597d7abef20dd5f89b16e4616233f02760b9d037
src/tests/slave_tests.cpp 628b0d0fc862264c9553e1660c7df548df9cd4a1
Diff: https://reviews.apache.org/r/65449/diff/5/
Changes: https://reviews.apache.org/r/65449/diff/4-5/
Testing
-------
make check
Dedicated test in #65448
Thanks,
Meng Zhu
