----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37168/#review123903 -----------------------------------------------------------
src/examples/dynamic_reservation_framework.cpp (line 32) <https://reviews.apache.org/r/37168/#comment186194> Looks like this can be removed? src/examples/dynamic_reservation_framework.cpp (line 119) <https://reviews.apache.org/r/37168/#comment186195> Could you clarify for me how the slave gets into the RESERVED state the first time? It seems to me we should have something here similar to what's in the switch for `State::UNRESERVING`, i.e., check the offer and see if the reserved resources are contained in it? The framework test passes for me though, so it seems to be working nonetheless? - Greg Mann On March 16, 2016, 6:32 a.m., Klaus Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37168/ > ----------------------------------------------------------- > > (Updated March 16, 2016, 6:32 a.m.) > > > Review request for mesos, Greg Mann, Joerg Schad, and Michael Park. > > > Bugs: MESOS-3063 > https://issues.apache.org/jira/browse/MESOS-3063 > > > Repository: mesos > > > Description > ------- > > Provide example for dynamic reservation features. > > > Diffs > ----- > > src/Makefile.am 9dd21b56af0500f7125b07bf535b45fe5c544aaf > src/examples/dynamic_reservation_framework.cpp PRE-CREATION > src/tests/dynamic_reservation_framework_test.sh PRE-CREATION > src/tests/examples_tests.cpp 6ddac17bb2ac0330bcc09eaab975ae70e84a7695 > > Diff: https://reviews.apache.org/r/37168/diff/ > > > Testing > ------- > > make > make check > > > Thanks, > > Klaus Ma > >
