----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27769/#review60812 -----------------------------------------------------------
Ship it! module the fix for duplicate task ids, which i'm assuming you'll mark as "blocks" this one. src/master/master.cpp <https://reviews.apache.org/r/27769/#comment102222> How about // Authorize each task. A task is in 'pendingTasks' // before it is authorized. - Vinod Kone On Nov. 10, 2014, 11:32 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/27769/ > ----------------------------------------------------------- > > (Updated Nov. 10, 2014, 11:32 p.m.) > > > Review request for mesos, Ben Mahler and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > Perform validation while launching each task. In that way, we can simplify a > few Task validators. Also, it will simply the validator for persistent disk > resources. > > > Diffs > ----- > > src/master/master.hpp 687f1789712dcd867b411badd85f4a12ae8f16d7 > src/master/master.cpp 0e182a6328e089bcc7e9b173e267f6d5ca357890 > > Diff: https://reviews.apache.org/r/27769/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >