----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15853/#review29473 -----------------------------------------------------------
I've sent out https://reviews.apache.org/r/15872 to address the abort() bug. src/sched/sched.cpp <https://reviews.apache.org/r/15853/#comment56707> Should this be a VLOG? Does this deserve us to leak a WARNING log message? src/sched/sched.cpp <https://reviews.apache.org/r/15853/#comment56706> Do we really need to log the framework id? src/sched/sched.cpp <https://reviews.apache.org/r/15853/#comment56705> Do we really need to log the framework id? - Ben Mahler On Nov. 26, 2013, 4:10 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15853/ > ----------------------------------------------------------- > > (Updated Nov. 26, 2013, 4:10 a.m.) > > > Review request for mesos, Benjamin Hindman and Ben Mahler. > > > Bugs: MESOS-551 > https://issues.apache.org/jira/browse/MESOS-551 > > > Repository: mesos-git > > > Description > ------- > > Couldn't update the review at https://reviews.apache.org/r/12603/ :( > > So creating a new one. > > > Diffs > ----- > > src/sched/sched.cpp af57f284b715e1498fd4b2a317706bc33792dec3 > src/tests/status_update_manager_tests.cpp > 184cd0eacd2e933998cec1ba30918b9b1c56c247 > > Diff: https://reviews.apache.org/r/15853/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
