----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35157/#review87668 -----------------------------------------------------------
Patch looks great! Reviews applied: [35164, 35157] All tests passed. - Mesos ReviewBot On June 12, 2015, 12:21 a.m., Bartek Plotka wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35157/ > ----------------------------------------------------------- > > (Updated June 12, 2015, 12:21 a.m.) > > > Review request for mesos, Ben Mahler, Jie Yu, Niklas Nielsen, Szymon Konefal, > and Vinod Kone. > > > Repository: mesos > > > Description > ------- > > Added tests for QoS Controller and ResourceEstimator fetching ResoruceUsage > from ResourceMonitor. > > This is the unit test for https://reviews.apache.org/r/34980/ and > https://reviews.apache.org/r/35164/ > > > Diffs > ----- > > include/mesos/type_utils.hpp 65668a6ba8524a3bfd241f72302f64f408163540 > src/common/type_utils.cpp e92f6f36de0955784619029a016667b46bbe221b > src/tests/mesos.hpp e19ef984f9e4696bd405027d6f19756cf23d0df2 > src/tests/mesos.cpp 5e574c5aa79d548e5730db021b0e8b77d27f220b > src/tests/oversubscription_tests.cpp > e8ae053dd9cd712e49bd2830e414b7a3d99c20ca > > Diff: https://reviews.apache.org/r/35157/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Bartek Plotka > >