----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37194/#review94463 -----------------------------------------------------------
Patch looks great! Reviews applied: [36929, 36930, 36954, 36956, 37054, 37055, 37091, 37105, 37142, 37159, 37194] All tests passed. - Mesos ReviewBot On Aug. 6, 2015, 6:19 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37194/ > ----------------------------------------------------------- > > (Updated Aug. 6, 2015, 6:19 p.m.) > > > Review request for mesos, Ian Downes, Timothy Chen, Vinod Kone, and Jiang Yan > Xu. > > > Repository: mesos > > > Description > ------- > > Fixed a bug in the MesosContainerizer creation logic. I did a few small style > fixes and added a few more comments in the patch as well. > > The bug is that currently we only put filesystem isolator to the front if > it's coming from modules. > > > Diffs > ----- > > src/slave/containerizer/mesos/containerizer.cpp > 6d07ff151770bac4eeeb7cd8c9d03f54f2e78ec1 > > Diff: https://reviews.apache.org/r/37194/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
