Re: Review Request 57109: Re-checkpointed the `Executor`s and `Task`s during agent recovery.

2017-03-03 Thread Michael Park
> On March 2, 2017, 3:11 p.m., Benjamin Mahler wrote: > > src/slave/slave.hpp > > Lines 1062 (patched) > > > > > > maybe `recheckpointExecutor`? Took this recommendation and also updated to `recheckpointTask`. >

Re: Review Request 57109: Re-checkpointed the `Executor`s and `Task`s during agent recovery.

2017-03-03 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57109/ --- (Updated March 3, 2017, 1:12 a.m.) Review request for mesos and Benjamin

Re: Review Request 57109: Re-checkpointed the `Executor`s and `Task`s during agent recovery.

2017-03-02 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57109/#review167758 --- Fix it, then Ship it! src/slave/slave.hpp Lines 416-417

Re: Review Request 57109: Re-checkpointed the `Executor`s and `Task`s during agent recovery.

2017-03-02 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57109/ --- (Updated March 2, 2017, 1:48 p.m.) Review request for mesos and Benjamin

Re: Review Request 57109: Re-checkpointed the `Executor`s and `Task`s during agent recovery.

2017-02-28 Thread Benjamin Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57109/#review167169 --- src/slave/slave.cpp (lines 5311 - 5315)

Review Request 57109: Re-checkpointed the `Executor`s and `Task`s during agent recovery.

2017-02-27 Thread Michael Park
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57109/ --- Review request for mesos and Benjamin Mahler. Bugs: MESOS-7061