Re: Review Request 44537: Update PendingUnavailabilityTest to use the new scheduler mock.

2016-03-10 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44537/#review123065 --- Ship it! Ship It! - Joris Van Remoortere On March 10, 2016,

Re: Review Request 44537: Update PendingUnavailabilityTest to use the new scheduler mock.

2016-03-10 Thread Anand Mazumdar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44537/#review122992 --- Ship it! - Anand Mazumdar On March 10, 2016, 9:08 p.m., Josep

Re: Review Request 44537: Update PendingUnavailabilityTest to use the new scheduler mock.

2016-03-10 Thread Joseph Wu
> On March 8, 2016, 5:54 p.m., Guangya Liu wrote: > > src/tests/master_maintenance_tests.cpp, lines 501-503 > > > > > > @Joseph, just a question, how can this guarantee that there is no other > > events after the Cl

Re: Review Request 44537: Update PendingUnavailabilityTest to use the new scheduler mock.

2016-03-10 Thread Joseph Wu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44537/ --- (Updated March 10, 2016, 1:08 p.m.) Review request for mesos, Anand Mazumdar, A