> On May 15, 2014, 12:40 a.m., Ben Mahler wrote: > > src/slave/slave.cpp, line 559 > > <https://reviews.apache.org/r/21464/diff/2/?file=582039#file582039line559> > > > > I think we need to update 'authenticate' to deal with the fact that it > > can now be invoked asynchronously. Namely, it needs to check whether we > > still need to authenticate, much like doReliableRegistration checks if we > > still need to register.
as discussed, restricted the scope of this patch to only do backoff for (re-)registration and not authentication (added a TODO). - Vinod ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21464/#review43057 ----------------------------------------------------------- On May 14, 2014, 11:06 p.m., Vinod Kone wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21464/ > ----------------------------------------------------------- > > (Updated May 14, 2014, 11:06 p.m.) > > > Review request for mesos, Ben Mahler and Jiang Yan Xu. > > > Bugs: MESOS-1276 > https://issues.apache.org/jira/browse/MESOS-1276 > > > Repository: mesos-git > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/slave/constants.hpp c0975254d51fc21dbf3bc006b3169886347f0a3f > src/slave/constants.cpp 1854b16f2c1ed2ae51d2061b0338c3eda4f3b403 > src/slave/flags.hpp 8616817b9602f6ccf56e2af61f36c1bd295df1a1 > src/slave/slave.hpp 29d2a1607a2b3c4443ea14d1cc2b61a065df3cad > src/slave/slave.cpp f8ed65bf7b7e4f3c0834c9e22a525137856e9b23 > src/tests/fault_tolerance_tests.cpp > 4796149beb10a6c668762f5efecd86520b809c42 > src/tests/master_tests.cpp 7aa678afc94869c8243485bd0604532dec43a1e2 > src/tests/mesos.cpp 242d98ad195a8ab1256918c48a2956e2e085d26d > src/tests/slave_recovery_tests.cpp 85c57b29f6a56683e0df9788dea64ebb36e00812 > src/tests/slave_tests.cpp dfbc6481a3a1c6fc234dea48bcfe14deab0bea86 > > Diff: https://reviews.apache.org/r/21464/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Vinod Kone > >
