> On Jan. 13, 2015, 11:29 p.m., Vinod Kone wrote: > > src/master/master.cpp, line 1337 > > <https://reviews.apache.org/r/29869/diff/1/?file=820049#file820049line1337> > > > > We should consider using 'OneOf' protobuf type to get this check for > > free. > > > > https://developers.google.com/protocol-buffers/docs/proto#oneof
That's a 2.6.0 feature: https://github.com/google/protobuf/releases/tag/v2.6.0 > On Jan. 13, 2015, 11:29 p.m., Vinod Kone wrote: > > src/master/master.cpp, lines 1380-1381 > > <https://reviews.apache.org/r/29869/diff/1/?file=820049#file820049line1380> > > > > Curious, why you want acceptOffers() call into this method instead of > > the current way. I imagine each operation to be implemented as a > > function/method, similar to what you did in Master::call(). Yep, that's what we're aiming for, we're thinking of running through the operations and creating a chain of computation. - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29869/#review67970 ----------------------------------------------------------- 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 > >
