----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9464/#review16745 -----------------------------------------------------------
src/slave/cgroups_isolation_module.cpp <https://reviews.apache.org/r/9464/#comment35565> period at the end. src/slave/cgroups_isolation_module.cpp <https://reviews.apache.org/r/9464/#comment35566> period. src/slave/cgroups_isolation_module.cpp <https://reviews.apache.org/r/9464/#comment35567> period src/slave/cgroups_isolation_module.cpp <https://reviews.apache.org/r/9464/#comment35570> We actually have a Duration abstraction to store time and easily convert to different units. Can you use that instead? also period at the end. src/slave/cgroups_isolation_module.cpp <https://reviews.apache.org/r/9464/#comment35574> Shouldn't this be a CHECK or LOG(FATAL)? src/slave/cgroups_isolation_module.cpp <https://reviews.apache.org/r/9464/#comment35577> thank you. src/slave/cgroups_isolation_module.cpp <https://reviews.apache.org/r/9464/#comment35579> If you change the check in initialize to LOG(FATAL), you can just do CHECK_SOME here. - Vinod Kone On Feb. 18, 2013, 11:51 p.m., David Mackey wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9464/ > ----------------------------------------------------------- > > (Updated Feb. 18, 2013, 11:51 p.m.) > > > Review request for mesos, Benjamin Hindman, Vinod Kone, and Ben Mahler. > > > Description > ------- > > Initial support for CPU "hard limits" via CFS Bandwidth Control in cgroups. > > CFS is unique relative to existing Mesos cgroups support in that it is a > "subfeature" of an already supported cgroups subsystem, cpu. Also, there are > two "tunables" for configuring CFS bandwidth limiting. > > There are 4 approaches one could take: > 1) Use the CFS bandwidth limiting if the feature is present > 2) Expose as separate flag, eg "cpu,cfs,memory,freezer" > 3) Add feature flag support to subsystems via an additional delimiter, eg > "cpu+cfs,memory,freezer". > 4) Add an additional control flag via some other means > > Option 2's downside breaks the 1:1 mapping between cgroups subsystems and a > cgroups resource flag. > Option 3's downside is it greatly increases complexity of parsing cgroups > subsystem flags. > > This diff takes option 1. > > > This addresses bug MESOS-315. > https://issues.apache.org/jira/browse/MESOS-315 > > > Diffs > ----- > > src/slave/cgroups_isolation_module.cpp 14f549e > > Diff: https://reviews.apache.org/r/9464/diff/ > > > Testing > ------- > > make check + additional testing > > > Thanks, > > David Mackey > >
