Re: Review Request 45798: Move long lived executor to use the v1 API.

2016-04-06 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45798/#review127526 --- Patch looks great! Reviews applied: [45795, 45796, 45797, 45798]

Re: Review Request 45798: Move long lived executor to use the v1 API.

2016-04-06 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45798/ --- (Updated April 6, 2016, 10:37 p.m.) Review request for mesos and Vinod Kone.

Re: Review Request 45798: Move long lived executor to use the v1 API.

2016-04-06 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45798/ --- (Updated April 6, 2016, 10:09 p.m.) Review request for mesos and Vinod Kone.

Re: Review Request 45798: Move long lived executor to use the v1 API.

2016-04-06 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45798/#review127425 --- Fix it, then Ship it! src/examples/long_lived_executor.cpp (li

Re: Review Request 45798: Move long lived executor to use the v1 API.

2016-04-05 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45798/#review127294 --- Patch looks great! Reviews applied: [45795, 45796, 45797, 45798]

Review Request 45798: Move long lived executor to use the v1 API.

2016-04-05 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45798/ --- Review request for mesos and Vinod Kone. Bugs: MESOS-4982 https://issues.ap