----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18370/#review35187 -----------------------------------------------------------
src/tests/cluster.hpp <https://reviews.apache.org/r/18370/#comment65630> typo: arguments src/tests/cluster.hpp <https://reviews.apache.org/r/18370/#comment65631> indent these lines two more as they're a continuation. src/tests/cluster.hpp <https://reviews.apache.org/r/18370/#comment65632> it seems odd that we're not setting allocatorProcess to the passed in allocatorProcess. If this is to extend the lifetime, maybe the allocator should own the allocator process? src/tests/cluster.hpp <https://reviews.apache.org/r/18370/#comment65635> CHECK the cast succeeds - Dominic Hamon On Feb. 21, 2014, noon, Charlie Carson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18370/ > ----------------------------------------------------------- > > (Updated Feb. 21, 2014, noon) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone. > > > Bugs: MESOS-1024 > https://issues.apache.org/jira/browse/MESOS-1024 > > > Repository: mesos-git > > > Description > ------- > > Refactor Cluster::Master::start methods > > There are currently two overloads of the Cluster::Masters::start > function. One takes a argument of AllocatorProcess* and the other > does not. The AllocatorProcess* overload serves two purposes: > 1) it allows an alternative implementation of AllocatorProcess to be > passed (i.e. a mock) > 2) it changes the destruction timing so that the passed in argument > can outlive the master. > > Beyond that, the two functions are identical. This changes the > parameter to be Option<AllocatorProcess*> and allows all the logic > to be in one method. The old function exists for back-compat but > now simply forward by passing in None() to the other function. > > This is for two purposes: > 1) reduce code duplication > 2) position the code so that we can also optionally pass in a mock > repaier. > > Review: https://reviews.apache.org/r/18370 > > > Diffs > ----- > > src/tests/cluster.hpp d1bf680ed3f6a0fb16af85a21409d653d44522ca > > Diff: https://reviews.apache.org/r/18370/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Charlie Carson > >
