----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20219/#review40070 -----------------------------------------------------------
src/slave/slave.hpp <https://reviews.apache.org/r/20219/#comment72887> You have in the description of this review "an option type ... is needed when the executor info isn't known up-front and the executor resources therefore cannot be known until after the containerizer has launched the container." Something along these lines would be great for posterity as a comment here! ;) src/slave/slave.cpp <https://reviews.apache.org/r/20219/#comment72888> Can we move the CHECK_SOME down here? launchedTasks[task.task_id()] = t; CHECK_SOME(resources); resources = resources.get() + task.resources(); return t; src/slave/slave.cpp <https://reviews.apache.org/r/20219/#comment72889> Swap these two? - Benjamin Hindman On April 10, 2014, 4:21 p.m., Niklas Nielsen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20219/ > ----------------------------------------------------------- > > (Updated April 10, 2014, 4:21 p.m.) > > > Review request for mesos, Ian Downes and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > Yet another split out of the task-info RR (r19795/r18403) which turns > Executor::resources into an option type. It is needed when the > executor info isn't known up-front (the main motivation behind the > task-info RR) and the executor resources therefore cannot be known > until after the containerizer has launched the container. > > > Diffs > ----- > > src/slave/http.cpp 594032d > src/slave/slave.hpp 08f6005 > src/slave/slave.cpp cddb241 > > Diff: https://reviews.apache.org/r/20219/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Niklas Nielsen > >
