> On Jan. 22, 2014, 10:51 p.m., Jiang Yan Xu wrote: > > src/master/detector.cpp, lines 344-349 > > <https://reviews.apache.org/r/17172/diff/1-2/?file=434508#file434508line344> > > > > Just thought this would prevent us from introducing new labels because > > old version of detector fail here with if a new label is present in the > > group. > > > > Ignoring the unrecognized label allows the old detectors to function > > they are rolled back. > > > > Do we want that flexibility. > > > >
Discussed with Yan offline. This should be fine because the way we introduce new labels is by first rolling the detectors (slaves/schedulers) to understand the new label before rolling the contenders(masters). Also, since fetched() is trying to parse the data of the 'leader' i think it is better to fail here than silently ignore. - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17172/#review32488 ----------------------------------------------------------- On Jan. 22, 2014, 11:37 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17172/ > ----------------------------------------------------------- > > (Updated Jan. 22, 2014, 11:37 p.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 2 for MESOS-903. > > > Diffs > ----- > > include/mesos/mesos.proto 655f86757487ddbe551fdcf53eb793e773ecdd34 > src/master/constants.hpp 8498c9b4cf5db1f67e14554cb9916dcd3376932d > src/master/constants.cpp 0b7c9f7851b7b4ddbef7ed7fb6a59396d2f6d193 > src/master/detector.cpp cf337cfa598f019275fb299d354b674361a4aa68 > src/zookeeper/detector.cpp 2759a2fd31a6037e4e08ce270d36703cf89f5c73 > src/zookeeper/group.hpp 354229fe4f32d0d284e07819e7235edbea105c66 > src/zookeeper/group.cpp e21dc6f76f71bfe70bd83fa14d760f28c2cfa3d9 > > Diff: https://reviews.apache.org/r/17172/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
