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

(Updated Feb. 12, 2018, 12:17 p.m.)


Review request for mesos, Benjamin Mahler, Chun-Hung Hsiao, and Vinod Kone.


Changes
-------

Patch updated. Thanks for the comment!


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/4/

Changes: https://reviews.apache.org/r/65449/diff/3-4/


Testing
-------

make check
Dedicated test in #65448


Thanks,

Meng Zhu

Reply via email to