----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16045/#review29835 -----------------------------------------------------------
Ship it! LGTM but we should have a subsequent patch to interpret the negative numbers in leader election. (e.g. negative numbers are excluded when looking for the minimum unless every sequence number in the group is negative) - Jiang Yan Xu On Dec. 5, 2013, 9:40 p.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16045/ > ----------------------------------------------------------- > > (Updated Dec. 5, 2013, 9:40 p.m.) > > > Review request for mesos, Benjamin Hindman, Vinod Kone, and Jiang Yan Xu. > > > Bugs: MESOS-867 > https://issues.apache.org/jira/browse/MESOS-867 > > > Repository: mesos-git > > > Description > ------- > > See MESOS-867 for context. > > > Diffs > ----- > > src/zookeeper/group.hpp 27b2ee9433e784ccc5b8809e8e0efe449e1da4ba > src/zookeeper/group.cpp 2ddc65e8655280cc7ff8e23aca561c6ec6e7e1e2 > > Diff: https://reviews.apache.org/r/16045/diff/ > > > Testing > ------- > > make check, also verified that numify<int32_t> works as expected. > > > Thanks, > > Ben Mahler > >
