----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34980/#review86366 -----------------------------------------------------------
Patch looks great! Reviews applied: [34962, 34980] All tests passed. - Mesos ReviewBot On June 3, 2015, 5:56 a.m., Bartek Plotka wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34980/ > ----------------------------------------------------------- > > (Updated June 3, 2015, 5:56 a.m.) > > > Review request for mesos, Jie Yu, Niklas Nielsen, Szymon Konefal, and Vinod > Kone. > > > Bugs: MESOS-2773 > https://issues.apache.org/jira/browse/MESOS-2773 > > > Repository: mesos > > > Description > ------- > > Pass callback to the resource estimator to retrieve ResourceUsage from > Resource Monitor on demand. > > This is neccessary, since ResourceEstimator needs data (current statistics > for each executor) on which it will base its slack predictions. > > > Diffs > ----- > > include/mesos/slave/resource_estimator.hpp > e7f5decdabdfc75587d784a08a98c62ffdfc2909 > src/slave/resource_estimators/noop.hpp > 2234ed6da17346c74c2e232ff9eae501531515e1 > src/slave/resource_estimators/noop.cpp PRE-CREATION > src/slave/slave.cpp 271cb03770cd08406054dfce35d0821475e49b05 > src/tests/mesos.hpp aeeed61ea987963345a8b5c6e09d14e51098bb2e > src/tests/oversubscription_tests.cpp > f047b90d8f0b4dfe7e6cad6dc58909451e9f894b > > Diff: https://reviews.apache.org/r/34980/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Bartek Plotka > >