----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61818/#review183635 -----------------------------------------------------------
Patch looks great! Reviews applied: [61811, 59746, 61818] Logs available here: http://104.210.40.105/logs/master/61818 - Mesos Reviewbot Windows On Aug. 22, 2017, 5:18 p.m., Alexander Rukletsov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61818/ > ----------------------------------------------------------- > > (Updated Aug. 22, 2017, 5:18 p.m.) > > > Review request for mesos, Gilbert Song, Ian Downes, and Jie Yu. > > > Repository: mesos > > > Description > ------- > > When the container launch is aborted and the container is > cleaned up in the containerizer before `executorLaunched()` is > invoked, `containerizer->wait()` called in `executorLaunched()` > does not hold a termination object. This case should not be > logged as an error. > > > Diffs > ----- > > src/slave/slave.cpp 50d2a10cd68f6611efd4e691e5325e6e0c06f33a > > > Diff: https://reviews.apache.org/r/61818/diff/1/ > > > Testing > ------- > > None: not a functional change. > > > Thanks, > > Alexander Rukletsov > >
