----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29869/#review67976 -----------------------------------------------------------
src/master/master.cpp <https://reviews.apache.org/r/29869/#comment112065> Event? Why this is related to event? src/master/master.cpp <https://reviews.apache.org/r/29869/#comment112066> Consider s/acceptOffers/call ? Having both the 'accept' parameter and 'acceptOffers' in the function name seems to be redundent? Also, using a unified name 'call' makes it clear that it's for the new API. src/master/master.cpp <https://reviews.apache.org/r/29869/#comment112067> I would suggest leaving empty with an Error message here. It'll be replaced anyway. - Jie Yu On Jan. 13, 2015, 11:12 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29869/ > ----------------------------------------------------------- > > (Updated Jan. 13, 2015, 11:12 p.m.) > > > Review request for mesos, Jie Yu, Michael Park, and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > This is an initial stub that only handles accepting offers. > > The plan (per the TODO) is to have a single path for launching tasks through > `acceptOffers`, as opposed to `launchTasks`. > > > Diffs > ----- > > src/master/master.hpp 26116aff1e965501c8d94ea0b5bd1be37f944887 > src/master/master.cpp 63ca19ab9618feccd93a2335f9287122a4665c5e > > Diff: https://reviews.apache.org/r/29869/diff/ > > > Testing > ------- > > The code path currently cannot be executed. Tests will be added once the > scheduler driver supports accepting offers. > > make check > > > Thanks, > > Ben Mahler > >
