----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10161/#review18529 -----------------------------------------------------------
will wait for a test(s) before giving a shipit. src/detector/detector.cpp <https://reviews.apache.org/r/10161/#comment38838> isn't this redundant? - Vinod Kone On March 28, 2013, 6:25 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10161/ > ----------------------------------------------------------- > > (Updated March 28, 2013, 6:25 p.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > This is also along the way to fixing MESOS-305. > > This fixes a bug where we were not clearing the sequence number when sending > a NoMasterDetectedMessage. > As a result, this removes the need for hiding NoMasterDetectedMessages from > contending non-leaders. > > > Diffs > ----- > > src/detector/detector.cpp 7a8355162d543e017505dd58efd2d7bf96f99623 > > Diff: https://reviews.apache.org/r/10161/diff/ > > > Testing > ------- > > make check > > I'd like to add a test for this, but I don't see how to induce a timedout() > call on the ZooKeeperMasterDetectorProcess, unless I add some plumbing to get > a handle to the underlying process. Suggestions? > > > Thanks, > > Ben Mahler > >
