> On Aug. 13, 2014, 2:42 a.m., Niklas Nielsen wrote: > > src/master/master.cpp, lines 2252-2253 > > <https://reviews.apache.org/r/24576/diff/1/?file=658113#file658113line2252> > > > > It this comment still relevant?
Good catch, I think vinod meant stack instead of heap, since they have always been on the heap. I will update it! - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24576/#review50396 ----------------------------------------------------------- On Aug. 11, 2014, 11: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, 11: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 > >