----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37168/#review97232 -----------------------------------------------------------
Just a bunch of high-level questions here. It looks like you followed the pattern for `src/examples/test_framework.cpp`. I think it would be probably better to follow `src/examples/persistent_volume_framework.cpp` since 1. It's likely that people will visit dynamic reservation and persistent volumes together. 2. There are many components (e.g. authorization, implicit status update acknowledgements, etc) included in the test_framework that are there for demo purposes. I think it would be better to keep this example's scope to dynamic reservation, and it should lead us to a simpler example. Also, do we need the custom executor for some reason? It doesn't seem to be doing much currently? Another high-level question is, how would you describe what `State` represents? It seems like it's associated with every offer, would it makes more sense to associate it with a task instead? - Michael Park On Aug. 28, 2015, 3:31 a.m., Klaus Ma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37168/ > ----------------------------------------------------------- > > (Updated Aug. 28, 2015, 3:31 a.m.) > > > Review request for mesos and Michael Park. > > > Bugs: MESOS-3063 > https://issues.apache.org/jira/browse/MESOS-3063 > > > Repository: mesos > > > Description > ------- > > MESOS-3063 (Add an example framework using dynamic reservation) > > > Diffs > ----- > > src/Makefile.am 7b620ff > src/examples/dynamic_reservation_executor.cpp PRE-CREATION > src/examples/dynamic_reservation_framework.cpp PRE-CREATION > src/tests/dynamic_reservation_framework_test.sh PRE-CREATION > src/tests/examples_tests.cpp 3f56b30 > src/tests/flags.hpp 3644956 > src/tests/script.cpp bcc1fab > > Diff: https://reviews.apache.org/r/37168/diff/ > > > Testing > ------- > > make > make check > > > Thanks, > > Klaus Ma > >