----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47082/#review136774 -----------------------------------------------------------
src/master/master.cpp (line 6858) <https://reviews.apache.org/r/47082/#comment201862> `nullptr` src/tests/master_tests.cpp (line 1807) <https://reviews.apache.org/r/47082/#comment201870> I wonder if `Times(0)` would be equivalent and more concise here? Can we have a new test case (maybe more than one?) to cover some of the corner cases here -- e.g., that we expect to see `LostSlaveMessage` when a framework isn't running any tasks on an agent but has a pending offer or reserved resource for that agent. We should also update the docs to describe this behavior. - Neil Conway On June 9, 2016, 1:08 a.m., Anindya Sinha wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47082/ > ----------------------------------------------------------- > > (Updated June 9, 2016, 1:08 a.m.) > > > Review request for mesos and Jiang Yan Xu. > > > Bugs: MESOS-5143 > https://issues.apache.org/jira/browse/MESOS-5143 > > > Repository: mesos > > > Description > ------- > > When a slave is removed, master sends a LostSlaveMessage to affected > frameworks only (instead of all registered frameworks). An affected > framework is a framework which satisfied one or more conditions of > the following: > > 1. There are tasks on this slave belonging to the framework. > 2. There are pending tasks on this slave belonging to the framework. > 3. Reserved resources on the slave have a matching role with the > role of the framework. > 4. There are pending offers or pending inverse offers from this slave > for the framework. > > > Diffs > ----- > > src/master/master.hpp 790da3ce686401c378ad9c62d497d60893c4ce41 > src/master/master.cpp 21ec70491ce6b79be57ff8db51d4b2fa682b32ce > src/tests/master_tests.cpp 34be015aa314a7574e9065efb7b1bb8e1570c5b7 > > Diff: https://reviews.apache.org/r/47082/diff/ > > > Testing > ------- > > All existing and modified tests passed. > > > Thanks, > > Anindya Sinha > >
