----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52756/#review152414 -----------------------------------------------------------
src/executor/executor.cpp (line 731) <https://reviews.apache.org/r/52756/#comment221416> I think this should call `_shutdown()` to ensure ShutdownProcess is spawned whenever we want to shutdown. - Vinod Kone On Oct. 12, 2016, 12:15 a.m., Anand Mazumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52756/ > ----------------------------------------------------------- > > (Updated Oct. 12, 2016, 12:15 a.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-6370 > https://issues.apache.org/jira/browse/MESOS-6370 > > > Repository: mesos > > > Description > ------- > > Previously, the executor library used to commit suicide after the > recovery timeout without invoking the executor's shutdown callback. > This behavior was not consistent with the executor driver. > > > Diffs > ----- > > src/executor/executor.cpp a0e5d8382fa3886b747ae92507dce17e75c2d749 > > Diff: https://reviews.apache.org/r/52756/diff/ > > > Testing > ------- > > Manual testing (It's hard to test this directly owing to the executor library > immediately invoking the `connected()` callback since the libprocess instance > initialized in tests is always active!) > > Filed MESOS-6373 to address this. > > > Thanks, > > Anand Mazumdar > >