Re: Review Request 41848: Do not enable task and executor run on different resources.

2016-03-11 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41848/ --- (Updated 三月 11, 2016, 9:12 a.m.) Review request for mesos, Ben Mahler, Artem Ha

Re: Review Request 41848: Do not enable task and executor run on different resources.

2016-01-22 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41848/ --- (Updated 一月 23, 2016, 6:05 a.m.) Review request for mesos, Ben Mahler, Artem Ha

Re: Review Request 41848: Do not enable task and executor run on different resources.

2016-01-13 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41848/ --- (Updated 一月 13, 2016, 2:28 p.m.) Review request for mesos, Ben Mahler, Artem Ha

Re: Review Request 41848: Do not enable task and executor run on different resources.

2016-01-07 Thread Guangya Liu
> On 一月 8, 2016, 2:02 a.m., Jian Qiu wrote: > > maybe should also add a test case with mix of usage slack and allocation > > slack? Yes. - Guangya --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.o

Re: Review Request 41848: Do not enable task and executor run on different resources.

2016-01-07 Thread Guangya Liu
> On 一月 7, 2016, 11:38 p.m., Joseph Wu wrote: > > src/master/validation.cpp, line 487 > > > > > > This is probably not necessary. > > > > Before you'd see: > > "Task (and its executor, if exists) uses bo

Re: Review Request 41848: Do not enable task and executor run on different resources.

2016-01-07 Thread Jian Qiu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41848/#review113389 --- maybe should also add a test case with mix of usage slack and alloc

Re: Review Request 41848: Do not enable task and executor run on different resources.

2016-01-07 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41848/#review113354 --- Overall, the changes and tests LGTM. src/master/validation.cpp (l

Re: Review Request 41848: Do not enable task and executor run on different resources.

2016-01-07 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41848/ --- (Updated 一月 7, 2016, 11:26 a.m.) Review request for mesos, Ben Mahler, Artem Ha

Re: Review Request 41848: Do not enable task and executor run on different resources.

2016-01-06 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41848/#review113212 --- src/master/validation.cpp (line 486)

Re: Review Request 41848: Do not enable task and executor run on different resources.

2016-01-03 Thread Guangya Liu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41848/ --- (Updated 一月 3, 2016, 9:26 a.m.) Review request for mesos, Ben Mahler, Artem Har