----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64521/#review193747 -----------------------------------------------------------
src/status_update_manager/status_update_manager_process.hpp Lines 168-172 (patched) <https://reviews.apache.org/r/64521/#comment272383> Could you submit a follow-up patch to update the comments in the offer operation SUM to explain this constraint? src/tests/offer_operation_status_update_manager_tests.cpp Lines 843 (patched) <https://reviews.apache.org/r/64521/#comment272387> s/latest status update/latest status/ I'll fix this while committing. - Greg Mann On Dec. 13, 2017, 12:25 a.m., Gaston Kleiman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64521/ > ----------------------------------------------------------- > > (Updated Dec. 13, 2017, 12:25 a.m.) > > > Review request for mesos, Chun-Hung Hsiao, Greg Mann, and Jie Yu. > > > Repository: mesos > > > Description > ------- > > Made `StatusUpdateManagerProcess` fill in the latest status update. > > > Diffs > ----- > > src/status_update_manager/status_update_manager_process.hpp > 1ac64410594134ce3e709d396d0a74a3d0aa2007 > src/tests/offer_operation_status_update_manager_tests.cpp > a5327d3d070a390f2ceb918dc53a024fbfe8777e > > > Diff: https://reviews.apache.org/r/64521/diff/3/ > > > Testing > ------- > > Added a new test, which passes on GNU/Linux. The other existing tests also > pass. > > > Thanks, > > Gaston Kleiman > >