----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55611/#review162038 -----------------------------------------------------------
Fix it, then Ship it! src/slave/slave.cpp (lines 557 - 560) <https://reviews.apache.org/r/55611/#comment233293> Should we add a test that fails without this patch, i.e. one in which the agent drops a status update which is generated during agent initialization/recovery? - Greg Mann On Jan. 17, 2017, 11:22 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55611/ > ----------------------------------------------------------- > > (Updated Jan. 17, 2017, 11:22 a.m.) > > > Review request for mesos and Greg Mann. > > > Repository: mesos > > > Description > ------- > > This is done so that any updates that the status update manager > receives while the agent is in recovery (e.g., executor terminated) > are not un-necessarily forwarded to and dropped by the agent. > > > Diffs > ----- > > src/slave/slave.cpp ae183e6b8b40094b4764525a6d63164f210ef9d8 > > Diff: https://reviews.apache.org/r/55611/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
