----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17572/#review33371 -----------------------------------------------------------
Ship it! src/common/protobuf_utils.hpp <https://reviews.apache.org/r/17572/#comment62830> Any reason for the call above to use stringify, and this one to rely on the cast operator? src/common/type_utils.hpp <https://reviews.apache.org/r/17572/#comment62831> These scare me a bit. Should we write a reflection-based equality operator, or do we want specialized equality semantics? - Ben Mahler On Jan. 31, 2014, 12:50 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17572/ > ----------------------------------------------------------- > > (Updated Jan. 31, 2014, 12:50 a.m.) > > > Review request for mesos, Benjamin Hindman, Brenden Matthews, Niklas Nielsen, > and Jiang Yan Xu. > > > Bugs: MESOS-672 > https://issues.apache.org/jira/browse/MESOS-672 > > > Repository: mesos-git > > > Description > ------- > > This is part 2. 'createMasterInfo()' is used in the subsequent reviews. > > > Diffs > ----- > > src/common/protobuf_utils.hpp c83cbd693abf87089288d03f01ce06200a5d9245 > src/common/type_utils.hpp fe6bf71d689f7bfd8b6ae1b8fab9b2e76e28e7a8 > > Diff: https://reviews.apache.org/r/17572/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
