----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17173/#review32506 -----------------------------------------------------------
src/master/contender.cpp <https://reviews.apache.org/r/17173/#comment61366> We should be controlling and injecting the label here, after this point the code underneath has no idea that this is/was a MasterInfo, so using something like 'info' for the label doesn't make sense. src/zookeeper/contender.cpp <https://reviews.apache.org/r/17173/#comment61367> Did using Some(...) not work? - Benjamin Hindman On Jan. 22, 2014, 4:20 a.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17173/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2014, 4:20 a.m.) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, Brenden Matthews, and > Jiang Yan Xu. > > > Bugs: MESOS-903 > https://issues.apache.org/jira/browse/MESOS-903 > > > Repository: mesos-git > > > Description > ------- > > See summary. > > This is part 3 of MESOS-903. > > > Diffs > ----- > > src/master/contender.hpp 2a7e7c4380dfe130dbdb925c94a27441e843ece9 > src/master/contender.cpp 89e368bae0485b06c7ee1bc5eade49c2a2d19959 > src/master/master.cpp 008033e24b329623a3bbe865699bf0b555c0606d > src/tests/master_contender_detector_tests.cpp > 9cd576f21561a86588ee0ea8a8458a9371284d47 > src/zookeeper/contender.cpp d8a5201cf79269d4bf419c6375eeabfbd914c672 > > Diff: https://reviews.apache.org/r/17173/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
