----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21838/#review43861 -----------------------------------------------------------
src/tests/master_tests.cpp <https://reviews.apache.org/r/21838/#comment78133> s/registrationMessage/frameworkRegisteredMessage/ src/tests/master_tests.cpp <https://reviews.apache.org/r/21838/#comment78134> use protobuf::createTask() src/tests/master_tests.cpp <https://reviews.apache.org/r/21838/#comment78136> s/acknowledgementMessage/statusUpdateAcknowledgementMessage/ src/tests/master_tests.cpp <https://reviews.apache.org/r/21838/#comment78137> Hmm. Why is a terminal update handling any different? If it is for some future reason, lets add it then. This test is huge! src/tests/slave_tests.cpp <https://reviews.apache.org/r/21838/#comment78138> s/registrationMessage/frameworkRegisteredMessage/ src/tests/slave_tests.cpp <https://reviews.apache.org/r/21838/#comment78139> protobuf::createTask() src/tests/slave_tests.cpp <https://reviews.apache.org/r/21838/#comment78140> s/acknowledgementMessage/statusUpdateAcknowledgementMessage/ src/tests/slave_tests.cpp <https://reviews.apache.org/r/21838/#comment78141> s/acknowledgementMessage2/statusUpdateAcknowledgementMessage2/ src/tests/slave_tests.cpp <https://reviews.apache.org/r/21838/#comment78142> kill this. the expectation below subsumes this. - Vinod Kone On May 23, 2014, 5:07 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21838/ > ----------------------------------------------------------- > > (Updated May 23, 2014, 5:07 a.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Bugs: MESOS-1411 > https://issues.apache.org/jira/browse/MESOS-1411 > > > Repository: mesos-git > > > Description > ------- > > This is part 2 of MESOS-1411. > > This ensures that the master can handle status update acknowledgement > messages coming from the scheduler driver. For now, it simply forwards these > on to the slaves. Later we will update the master to rely the acknowledgement > messages to remove tasks from its memory (MESOS-1410). > > > Diffs > ----- > > src/master/master.hpp 99c70631ba10e8811ab0011881f30134c4d0373c > src/master/master.cpp e5d8edf4db932e2c48756df67750b02bbff59c55 > src/tests/master_tests.cpp b0e05de9a7226ee330007a3f3fe71976fb0748d6 > src/tests/slave_tests.cpp 85ca5c41bfafe4f932ea51b4fb9654b2fc828232 > > Diff: https://reviews.apache.org/r/21838/diff/ > > > Testing > ------- > > Added two tests here: > > (1) Send the acknowledgement through the Master by intercepting it when it > leaves the driver. > > (2) Ensure the slave ignores an acknowledgement from a non-leading master. > > Ran each test with 1000 iterations. > > > Thanks, > > Ben Mahler > >
