----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35911/#review89651 -----------------------------------------------------------
Ship it! Ship It! - Vinod Kone On June 27, 2015, 12:36 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35911/ > ----------------------------------------------------------- > > (Updated June 27, 2015, 12:36 a.m.) > > > Review request for mesos, Jie Yu and Vinod Kone. > > > Bugs: MESOS-2940 > https://issues.apache.org/jira/browse/MESOS-2940 > > > Repository: mesos > > > Description > ------- > > The `TaskStatus` contains an optional `UUID`, which should be unset for > master-generated updates. > > This udpates the `StatusUpdate` 'uuid' to also be optional. > > Since the master and slave assume the 'uuid' is set, this adds CHECKs for > now. Later, we'll want to properly validate this instead. I left a TODO for > this. > > > Diffs > ----- > > include/mesos/mesos.proto 81008ed85daea798ed19d1031de8421a7dc3fb19 > src/common/type_utils.cpp f88dff76a0dd57f5e16dc4d908655b4bb9d460a5 > src/master/master.cpp 0782b543b451921d2240958c7ef612a9e30972df > src/messages/messages.proto 1c8d79e3fca365520cdd67051f8730593955cab6 > src/sched/sched.cpp bc76c71ae9d44bdddd291049223366e38cb0fd0c > src/scheduler/scheduler.cpp 1efc6fb351e49deaa8f626823592bc9155f5137b > src/slave/slave.cpp b3e1ccc28d2698d8bc91829d70787dc2fc17b80d > src/slave/status_update_manager.cpp > 0ad24500c3007202263794fd094fa60535d8f58c > > Diff: https://reviews.apache.org/r/35911/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Ben Mahler > >