----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38051/#review100394 -----------------------------------------------------------
Patch looks great! Reviews applied: [38051] All tests passed. - Mesos ReviewBot On Sept. 24, 2015, 9:45 a.m., Yong Qiao Wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38051/ > ----------------------------------------------------------- > > (Updated Sept. 24, 2015, 9:45 a.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-2864 > https://issues.apache.org/jira/browse/MESOS-2864 > > > Repository: mesos > > > Description > ------- > > Only update the task status when its old status is not terminal. > > > Diffs > ----- > > src/master/master.cpp 6bee4f3 > src/tests/status_update_manager_tests.cpp 9970d71 > > Diff: https://reviews.apache.org/r/38051/diff/ > > > Testing > ------- > > UT: > 1. Write a test for this change. > 2. make successfully! > 3. make check successfully! > 4. Run test framework successfully! > > > Thanks, > > Yong Qiao Wang > >