Re: Review Request 66462: Added new operation states to be used for status reconciliation.

2018-04-20 Thread Gaston Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66462/ --- (Updated April 20, 2018, 3:53 p.m.) Review request for mesos and Greg Mann.

Re: Review Request 66462: Added new operation states to be used for status reconciliation.

2018-04-12 Thread Gaston Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66462/ --- (Updated April 12, 2018, 9:59 a.m.) Review request for mesos and Greg Mann.

Re: Review Request 66462: Added new operation states to be used for status reconciliation.

2018-04-12 Thread Gaston Kleiman
> On April 11, 2018, 7:06 p.m., Chun-Hung Hsiao wrote: > > Are all of these necessary for now? Yeah, they are all generated/sent by the {{ReconcileOperations}} handler. > On April 11, 2018, 7:06 p.m., Chun-Hung Hsiao wrote: > > src/slave/slave.cpp > > Lines 8034-8038 (patched) > >

Re: Review Request 66462: Added new operation states to be used for status reconciliation.

2018-04-11 Thread Chun-Hung Hsiao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66462/#review200957 --- Are all of these necessary for now? src/slave/slave.cpp Lines

Re: Review Request 66462: Added new operation states to be used for status reconciliation.

2018-04-11 Thread Gaston Kleiman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66462/ --- (Updated April 11, 2018, 10:58 a.m.) Review request for mesos and Greg Mann.

Re: Review Request 66462: Added new operation states to be used for status reconciliation.

2018-04-10 Thread Greg Mann
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66462/#review200834 --- Fix it, then Ship it! include/mesos/mesos.proto Lines 2314