----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8128/#review13588 -----------------------------------------------------------
src/master/master.cpp <https://reviews.apache.org/r/8128/#comment29161> Why not always forward this to the scheduler, irrespective of whether the master knows about slaveId/ executorId/frameworkId? src/sched/sched.cpp <https://reviews.apache.org/r/8128/#comment29159> s/on/on slave/ src/sched/sched.cpp <https://reviews.apache.org/r/8128/#comment29160> we are trying to use a new formatting for function calls. if it fits on the 2nd line. scheduler->executorLost( driver, executorId, slaveId, static_cast<int>(status); if it doesn't scheduler->executorLost( driver, executorId, slaveId, static_cast<int>(status)) - Vinod Kone On Nov. 19, 2012, 9:14 p.m., Charles Reiss wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8128/ > ----------------------------------------------------------- > > (Updated Nov. 19, 2012, 9:14 p.m.) > > > Review request for mesos and Benjamin Hindman. > > > Description > ------- > > Forward ExitedExecutorMessages from the master to framework schedulers, and > call Scheduler::executorLost when they are received. > > > This addresses bug MESOS-313. > https://issues.apache.org/jira/browse/MESOS-313 > > > Diffs > ----- > > src/master/master.cpp 87fad0e > src/sched/sched.cpp a1247f9 > src/tests/gc_tests.cpp 6b3ba02 > src/tests/master_tests.cpp 948ab5d > > Diff: https://reviews.apache.org/r/8128/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Charles Reiss > >
