----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36106/#review91343 -----------------------------------------------------------
src/linux/cgroups.hpp (line 438) <https://reviews.apache.org/r/36106/#comment144699> benm: Remove 1 space. src/linux/cgroups.cpp (line 2003) <https://reviews.apache.org/r/36106/#comment144700> benm: s/std::string/string/g src/linux/cgroups.cpp (line 2004) <https://reviews.apache.org/r/36106/#comment144701> ditto src/linux/cgroups.cpp (line 2022) <https://reviews.apache.org/r/36106/#comment144698> benm: s/Error getting/Failed to get/g Also we should use ErrnoError as well. src/linux/cgroups.cpp (line 2030) <https://reviews.apache.org/r/36106/#comment144702> benm: All the error messages in this file follows the "Failed to..." instead of "Error..." message, we should be consistent. src/tests/cgroups_tests.cpp (line 62) <https://reviews.apache.org/r/36106/#comment144697> benm: Remove alias src/tests/cgroups_tests.cpp (line 1199) <https://reviews.apache.org/r/36106/#comment144695> benm: Remove storing result and just ASSERT_SOME(cgroups::cpuacct::stat(hierarchy, TEST_CGROUPS_ROOT)); src/tests/cgroups_tests.cpp (line 1203) <https://reviews.apache.org/r/36106/#comment144696> benm: AWAIT_READY(cgroups::destroy(hierarchy, TEST_CGROUPS_ROOT)); - Timothy Chen On July 10, 2015, 8:47 p.m., Jojy Varghese wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/36106/ > ----------------------------------------------------------- > > (Updated July 10, 2015, 8:47 p.m.) > > > Review request for mesos, Ian Downes, Jie Yu, Joris Van Remoortere, and > Timothy Chen. > > > Bugs: MESOS-2961 > https://issues.apache.org/jira/browse/MESOS-2961 > > > Repository: mesos > > > Description > ------- > > cgroups implementation does not have a cpuacct subsystem implementation as of > today. Adding the implementation for stat function. > > Changes: > - added Stats class to encapsulate cpuacct.stat data > - added implementation for cpuacct::stats > - added unit tests > > Jira: MESOS-2961 > > > Diffs > ----- > > src/linux/cgroups.hpp 73b98317880eea3d6a2ba37ac56d1f7e3600ba94 > src/linux/cgroups.cpp 4c006d0c7382b940a83359d636c0d48952cdbb00 > src/tests/cgroups_tests.cpp 475f48a474eea708f98d8c0300862351a2d4379a > > Diff: https://reviews.apache.org/r/36106/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jojy Varghese > >