----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8238/#review13789 -----------------------------------------------------------
src/slave/cgroups_isolation_module.cpp <https://reviews.apache.org/r/8238/#comment29466> why the trim here and below? are you just being defensive? - Vinod Kone On Nov. 27, 2012, 11:15 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8238/ > ----------------------------------------------------------- > > (Updated Nov. 27, 2012, 11:15 p.m.) > > > Review request for mesos, Benjamin Hindman, Vinod Kone, and Jie Yu. > > > Description > ------- > > The trailing newline in the cgroup.cpus file caused the slave to fail during > CgroupIsolationModule initialization. > > > Diffs > ----- > > src/slave/cgroups_isolation_module.cpp > 0f2975d1adf874dba4d0a539eb5c99233cef6e6b > > Diff: https://reviews.apache.org/r/8238/diff/ > > > Testing > ------- > > make check + deployment > > > Thanks, > > Ben Mahler > >
