----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9146/#review16821 -----------------------------------------------------------
Ship it! src/slave/process_based_isolation_module.cpp <https://reviews.apache.org/r/9146/#comment35736> Move this down just below 'ProcessInfo* Info =' and above 'result.set_timestamp...'. src/slave/process_based_isolation_module.cpp <https://reviews.apache.org/r/9146/#comment35735> Please merge this with ifdef linux below. - Benjamin Hindman On Feb. 13, 2013, 2:55 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9146/ > ----------------------------------------------------------- > > (Updated Feb. 13, 2013, 2:55 a.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > This implements process isolation module resource collection for linux. > > My following change implements the OSX version. > > > This addresses bug MESOS-324. > https://issues.apache.org/jira/browse/MESOS-324 > > > Diffs > ----- > > src/slave/process_based_isolation_module.cpp > 3d50a4b652e4e09dd57e744e408c8fb79ff3fbf5 > > Diff: https://reviews.apache.org/r/9146/diff/ > > > Testing > ------- > > End-to-end via the webui. > > > Thanks, > > Ben Mahler > >
