----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20502/#review40985 -----------------------------------------------------------
src/slave/slave.cpp <https://reviews.apache.org/r/20502/#comment74333> Why use Duration::create instead of the built in * and + Duration operators? Duration next = Seconds(1) + backoff * (...); src/slave/slave.cpp <https://reviews.apache.org/r/20502/#comment74334> Do you need the Duration() wrapper? - Ben Mahler On April 20, 2014, 9:49 p.m., Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20502/ > ----------------------------------------------------------- > > (Updated April 20, 2014, 9:49 p.m.) > > > Review request for mesos, Ben Mahler and Vinod Kone. > > > Bugs: MESOS-982 > https://issues.apache.org/jira/browse/MESOS-982 > > > Repository: mesos-git > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/slave/slave.hpp 1e9879582315fabb76659e9f8eb03f90188fbfa0 > src/slave/slave.cpp d6ec87c2232c2172f471ae30711b5da1c7050570 > > Diff: https://reviews.apache.org/r/20502/diff/ > > > Testing > ------- > > make check. Also tested the random backoff logic separately. > > > Thanks, > > Jiang Yan Xu > >
