Re: Review Request 55611: Paused the status update manager in `Slave::initialize()`.

2017-01-18 Thread Vinod Kone
> On Jan. 18, 2017, 7:49 a.m., Greg Mann wrote: > > src/slave/slave.cpp, lines 557-560 > > > > > > Should we add a test that fails without this patch, i.e. one in which > > the agent drops a status update which is

Re: Review Request 55611: Paused the status update manager in `Slave::initialize()`.

2017-01-18 Thread Vinod Kone
> On Jan. 18, 2017, 7:49 a.m., Greg Mann wrote: > > src/slave/slave.cpp, lines 557-560 > > > > > > Should we add a test that fails without this patch, i.e. one in which > > the agent drops a status update which is

Re: Review Request 55611: Paused the status update manager in `Slave::initialize()`.

2017-01-17 Thread Greg Mann
--- 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)

Review Request 55611: Paused the status update manager in `Slave::initialize()`.

2017-01-17 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55611/ --- Review request for mesos and Greg Mann. Repository: mesos Description