> On May 16, 2015, 1:03 a.m., Niklas Nielsen wrote: > > src/slave/slave.cpp, line 4080 > > <https://reviews.apache.org/r/34299/diff/1/?file=961836#file961836line4080> > > > > This is being executed in the context of the estimator thread? Is this > > safe?
This will be executed in the slave's thread because we registered a `defer(self(), &Slave::receiveOversubscribedResources, lambda::_1)` with the resource estimator. - Jie ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34299/#review84011 ----------------------------------------------------------- On May 15, 2015, 11:38 p.m., Jie Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34299/ > ----------------------------------------------------------- > > (Updated May 15, 2015, 11:38 p.m.) > > > Review request for mesos, Joris Van Remoortere, Niklas Nielsen, and Vinod > Kone. > > > Bugs: MESOS-2735 > https://issues.apache.org/jira/browse/MESOS-2735 > > > Repository: mesos > > > Description > ------- > > Changed to use a push model for resource estimator. See ticket for details. > > This patch also makes the interval configurable through slave flags. > > > Diffs > ----- > > include/mesos/slave/resource_estimator.hpp > 363961541c9b49763e8cbafe982421d45516db0d > src/slave/constants.hpp df02043fcd1b5bffc13aa0683ad9b64372188300 > src/slave/constants.cpp 07f699a9e5c1b56d6e0435077823ad0f863ad63c > src/slave/flags.hpp ca7cc131f70d58982262fd29986d46877d563035 > src/slave/flags.cpp f35c76a342d03700710bb91bf4c523cc99769228 > src/slave/resource_estimator.hpp bdf62bab9b632aa16801e47978962a7ab24b53b4 > src/slave/resource_estimator.cpp 13d706c30d36c9f304acf288f02d6c9558bac781 > src/slave/slave.hpp b62ed7b207fa337027baffa1c75139641a41dad6 > src/slave/slave.cpp 132f83e05dd4113c8f6d20a6a64b12fb8f05dd36 > src/tests/mesos.hpp df8cd200a6970209588b63a1d58709b1f913a305 > src/tests/oversubscription_tests.cpp > 64c2ede1f50546f6de1ecb2972036b51fe17e358 > > Diff: https://reviews.apache.org/r/34299/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Jie Yu > >
