----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37466/#review96168 -----------------------------------------------------------
src/tests/containerizer/perf_tests.cpp (line 52) <https://reviews.apache.org/r/37466/#comment151466> I think this should still be in perf.hpp even though it is just for testing. - Cong Wang On Aug. 21, 2015, 4:06 a.m., Paul Brett wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37466/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2015, 4:06 a.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-2834 > https://issues.apache.org/jira/browse/MESOS-2834 > > > Repository: mesos > > > Description > ------- > > Update perf tests to including testing the supported perf output formats. > > > Diffs > ----- > > src/linux/perf.cpp cdc5f8314a875ababf2b17a32873775d808e1c78 > src/tests/containerizer/perf_tests.cpp > 6b3d70f3e7ea8f59f94e6961491d4e9a730e3334 > > Diff: https://reviews.apache.org/r/37466/diff/ > > > Testing > ------- > > sudo make check > > > Thanks, > > Paul Brett > >