----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19085/#review36925 -----------------------------------------------------------
src/master/registrar.hpp <https://reviews.apache.org/r/19085/#comment68112> an enum with STRICT/NONSTRICT would be clearer from an API point of view. src/master/registrar.cpp <https://reviews.apache.org/r/19085/#comment68114> this would also make it easier to add unit tests for each operation. src/master/registrar.cpp <https://reviews.apache.org/r/19085/#comment68113> no need for else as you return in the if. src/master/registrar.cpp <https://reviews.apache.org/r/19085/#comment68115> no need for else as you return in the if. src/master/registrar.cpp <https://reviews.apache.org/r/19085/#comment68116> no need for else as you return in the if. src/tests/registrar_tests.cpp <https://reviews.apache.org/r/19085/#comment68117> if we had AWAIT_TRUE and AWAIT_FALSE this would be cleaner. src/tests/registrar_tests.cpp <https://reviews.apache.org/r/19085/#comment68118> AWAIT_EQ(!strict, registrar.readmit(info2, strict)); - Dominic Hamon On March 11, 2014, 5:45 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19085/ > ----------------------------------------------------------- > > (Updated March 11, 2014, 5:45 p.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Bugs: MESOS-764 > https://issues.apache.org/jira/browse/MESOS-764 > > > Repository: mesos-git > > > Description > ------- > > This adds the notion of "non-strict" operations in the Registrar. Operations > that are not "strict" are always permitted. > > This allows the state to be bootstrapped from the current cluster state and > is essential for those upgrading a running cluster from a pre-Registrar > version of Mesos. > > I have included a TODO that discusses a simplified design of the Registrar > per a chat with benh. I'd like to implement this simplified design in a > subsequent change so I'd like to hear if there are any potential issues. The > new design would mean pushing Operations out to a header and allowing callers > to use Operations directly. > > > Diffs > ----- > > src/master/registrar.hpp 987a63b33bd26add3253f25f1471dd4a73471830 > src/master/registrar.cpp d30172cee8525a80661135e6554bc0d78c7e314d > src/tests/registrar_tests.cpp 95a6b53e479ea9131aca9b39f096e6dc9e6848f8 > > Diff: https://reviews.apache.org/r/19085/diff/ > > > Testing > ------- > > make check > > I've parameterized the existing tests for strict and non-strict operation. > I've also added a test to ensure that the state can be bootstrapped. > > > Thanks, > > Ben Mahler > >
