----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29976/#review68832 -----------------------------------------------------------
src/master/master.cpp <https://reviews.apache.org/r/29976/#comment113297> s/into a/into/ ? "offer operation validators use that...", what is "that" ? do you mean *some* of those standalone validators? src/master/master.cpp <https://reviews.apache.org/r/29976/#comment113308> s/reply/ACCEPT call/ src/master/master.cpp <https://reviews.apache.org/r/29976/#comment113305> // If LAUNCH operation is included authorize the tasks. A task ... src/master/master.cpp <https://reviews.apache.org/r/29976/#comment113307> The only weird part about _accept() is that it takes "_authorizations" which is specifici to LAUNCH operation. Maybe a comment explaining why it had to be done that way? src/master/master.cpp <https://reviews.apache.org/r/29976/#comment113313> Lets use this variable when offeredResources are actually being validated. Using a separate temp right now is a bit confusing. src/tests/resource_offers_tests.cpp <https://reviews.apache.org/r/29976/#comment113311> Any reason to not do this is a subsequent patch? - Vinod Kone On Jan. 20, 2015, 6:49 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29976/ > ----------------------------------------------------------- > > (Updated Jan. 20, 2015, 6:49 p.m.) > > > Review request for mesos, Ben Mahler, Michael Park, and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > Moved launchTasks logics to ACCEPT call handler in master. > > This patch moves all logics of launchTasks in master to the ACCEPT call > handler in master, and leaves stubs for other offer operations. We still > keeps the launchTasks handler for backwards compatibility. launchTasks will > simply create an Accept call message, and invoke the ACCEPT call handler. > > > Diffs > ----- > > src/master/master.hpp a8ce4d023ddea36cb83a2dc7b95abd12342f345a > src/master/master.cpp e9dcca3c92c94f3123519855e238bcef47eeece9 > src/tests/resource_offers_tests.cpp > d098e7016ac0da7f1d629af099bb1b8fa66da839 > > Diff: https://reviews.apache.org/r/29976/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
