Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master.

2016-02-17 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/ --- (Updated Feb. 18, 2016, 7:45 a.m.) Review request for mesos, Ben Mahler and

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master.

2016-02-12 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/#review115114 --- Bad patch! Reviews applied: [41302, 41305] Failed command:

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master.

2016-01-18 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/ --- (Updated Jan. 18, 2016, 10:19 p.m.) Review request for mesos, Ben Mahler,

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master.

2016-01-18 Thread Klaus Ma
> On Jan. 13, 2016, 8:39 p.m., Klaus Ma wrote: > > @Joseph/Vinod, would you help to review those patches move executor from > > slave to master? > > Joseph Wu wrote: > There are still several open issues open on each prior review. Can you > go through and make sure those are addressed? >

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master.

2016-01-18 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/#review115023 --- Bad patch! Reviews applied: [41302, 41305] Failed command:

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master.

2016-01-18 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/ --- (Updated Jan. 19, 2016, 10:55 a.m.) Review request for mesos, Ben Mahler,

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master.

2016-01-15 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/ --- (Updated Jan. 16, 2016, noon) Review request for mesos, Ben Mahler, Guangya

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master

2016-01-14 Thread Klaus Ma
> On Jan. 13, 2016, 8:39 p.m., Klaus Ma wrote: > > @Joseph/Vinod, would you help to review those patches move executor from > > slave to master? > > Joseph Wu wrote: > There are still several open issues open on each prior review. Can you > go through and make sure those are addressed?

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master

2016-01-14 Thread Joseph Wu
> On Jan. 13, 2016, 4:39 a.m., Klaus Ma wrote: > > @Joseph/Vinod, would you help to review those patches move executor from > > slave to master? There are still several open issues open on each prior review. Can you go through and make sure those are addressed? - Joseph

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master

2016-01-14 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/ --- (Updated Jan. 15, 2016, 1:19 p.m.) Review request for mesos, Ben Mahler,

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master

2016-01-14 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/#review114676 --- Bad patch! Reviews applied: [41302] Failed command:

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master

2016-01-13 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/#review114191 --- @Joseph/Vinod, would you help to review those patches move

Re: Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master

2015-12-13 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/#review110122 --- Patch looks great! Reviews applied: [41302, 41305, 41306, 41308]

Review Request 41308: MESOS-1718: Unit Test for moving getExecutorInfo from slave to master

2015-12-12 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41308/ --- Review request for mesos, Ben Mahler, Guangya Liu, Artem Harutyunyan, Ian