Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

> On Nov. 11, 2015, 9:28 p.m., Vinod Kone wrote: > > src/slave/slave.cpp, lines 4244-4247 > > > > > > why do it here instead of in recoverFramework() #4363? that feels more > > consistent with #1345. > > James

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40177/ --- (Updated Nov. 18, 2015, 6:49 p.m.) Review request for mesos, Kapil Arya and

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40177/#review107070 --- src/slave/slave.cpp (line 4321)

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40177/ --- (Updated Nov. 18, 2015, 9:56 p.m.) Review request for mesos, Kapil Arya and

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40177/#review107087 --- Patch looks great! Reviews applied: [40177] Passed command:

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

> On Nov. 11, 2015, 9:28 p.m., Vinod Kone wrote: > > src/slave/slave.cpp, lines 4244-4247 > > > > > > why do it here instead of in recoverFramework() #4363? that feels more > > consistent with #1345. > > James

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40177/#review106061 --- Ship it! Ship It! - Guangya Liu On 十一月 11, 2015, 5:59 a.m.,

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40177/ --- (Updated Nov. 12, 2015, 5:41 a.m.) Review request for mesos, Kapil Arya and

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40177/#review106200 --- Patch looks great! Reviews applied: [40177] All tests passed. -

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40177/#review106142 --- src/slave/slave.cpp (lines 4244 - 4247)

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40177/#review106018 --- Patch looks great! Reviews applied: [40177] All tests passed. -

Re: Review Request 40177: Re-checkpoint frameworks after agent recovery.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40177/#review106020 --- Patch looks great! Reviews applied: [40177] All tests passed. -