----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10161/ -----------------------------------------------------------
(Updated April 19, 2013, 8:54 p.m.) Review request for mesos, Benjamin Hindman and Vinod Kone. Changes ------- Vinod's review. 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 (updated) ----- src/detector/detector.hpp ed485bd86d2bff1046a234a5776c1081a4136bc5 src/detector/detector.cpp 7a8355162d543e017505dd58efd2d7bf96f99623 src/tests/zookeeper_tests.cpp 0855f5aee0baef22c4ecbed1b88f14f16bfff532 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
