----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24576/#review50396 -----------------------------------------------------------
Ship it! Modulo Vinod's comments. Looks good otherwise! src/master/master.cpp <https://reviews.apache.org/r/24576/#comment88172> It this comment still relevant? - Niklas Nielsen On Aug. 11, 2014, 4:32 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24576/ > ----------------------------------------------------------- > > (Updated Aug. 11, 2014, 4:32 p.m.) > > > Review request for mesos, Niklas Nielsen and Vinod Kone. > > > Repository: mesos-git > > > Description > ------- > > There were a few cleanups here that I did on the way to MESOS-1620. > > (1) Remove the need for Offer/Task Visitor memory cleanup by using Owned<>. > (2) Restructured the launch task code to be easier to read and understand. > > > Diffs > ----- > > src/master/master.cpp e688b41b9f2e555acd8fe0da5d3eb4e8bce32211 > > Diff: https://reviews.apache.org/r/24576/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Ben Mahler > >