----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37532/#review99061 -----------------------------------------------------------
include/mesos/scheduler.hpp (line 272) <https://reviews.apache.org/r/37532/#comment155898> Should we even bother adding this to the old API? Note that adding a new call to the driver will impose a upgrade dependency between schedulers and master (master first, libmesos for scheduler next, scheduler jar last) more importantly, add a comment here that scheduler should call reviveOffers() to resume getting offers. include/mesos/scheduler/scheduler.proto (line 173) <https://reviews.apache.org/r/37532/#comment155891> period at the end. also, does this fit within 80 chars? src/master/master.cpp (line 1747) <https://reviews.apache.org/r/37532/#comment155893> you also need to add a case statement to master/http.cpp. src/master/master.cpp (line 2542) <https://reviews.apache.org/r/37532/#comment155892> s/just ignore/ignoring/ - Vinod Kone On Sept. 5, 2015, 12:01 a.m., Guangya Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37532/ > ----------------------------------------------------------- > > (Updated Sept. 5, 2015, 12:01 a.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-3037 > https://issues.apache.org/jira/browse/MESOS-3037 > > > Repository: mesos > > > Description > ------- > > This is just part of MESOS-3037, this patch only add the interface > of QUIESCE call. > > > Diffs > ----- > > include/mesos/scheduler.hpp ee198b6955882f4f31466ca05429ca16fbf2f5cd > include/mesos/scheduler/scheduler.proto > 89daf8a6b74057ee156b3ad691397e76fcb835b8 > src/master/master.hpp e1331851c19e3372a4a525dcfd7ba2a01c3e97a6 > src/master/master.cpp 5589eca4317b597de509f3387cfc349083b361ac > src/sched/sched.cpp 012af0508eeceeccd168b29f36fa258d20b28c21 > > Diff: https://reviews.apache.org/r/37532/diff/ > > > Testing > ------- > > > Thanks, > > Guangya Liu > >