----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34980/#review86441 -----------------------------------------------------------
src/slave/slave.cpp <https://reviews.apache.org/r/34980/#comment138429> Hum, this looks buggy to me. You copy the 'monitor' instance, which means its internal member 'process' will be deleted twice. I think we might want to make ResourceMonitor a pointer and make sure its life time is longer than the resource estimator. - Jie Yu 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 > >
