----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68304/#review207255 -----------------------------------------------------------
Per offline conversation, seems like we could simplify this by modeling as: ``` MIN, MIN + factor*2^N ``` src/slave/slave.cpp Lines 1287-1290 (original), 1286-1289 (patched) <https://reviews.apache.org/r/68304/#comment290587> Shouldn't we specialize this between authentication and registration? - Benjamin Mahler On Aug. 14, 2018, 5:33 a.m., Meng Zhu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/68304/ > ----------------------------------------------------------- > > (Updated Aug. 14, 2018, 5:33 a.m.) > > > Review request for mesos, Benjamin Mahler and Gastón Kleiman. > > > Bugs: MESOS-9147 > https://issues.apache.org/jira/browse/MESOS-9147 > > > Repository: mesos > > > Description > ------- > > This patch fixed the backoff time calculation > overflow bug described in MESOS-9147. > > The old approach times out an authentication request after > 5s and then retries after some backoff time. This is not > optimal because, if the agent is going to backoff some time > before retry, we might as well wait that long for the > previous authentication request (instead of timeout early). > > This patch combines the authentication timeout and > authentication retry backoff interval into a single > wait time interval. Now agent will timeout the previous > authentication request after the wait time interval and > then immediately retry. > > > Diffs > ----- > > src/slave/constants.hpp 0bd9f371ca24df66055f31bca0d57625dc7642d9 > src/slave/slave.hpp 802d4eb9e9eba2f1175dd85e56dcc80a61e32f74 > src/slave/slave.cpp 78e8666f402be58af5b6e20a715da4998af2615c > src/tests/authentication_tests.cpp c9a8f85951a50e278ae509f4efa7105755015ce9 > > > Diff: https://reviews.apache.org/r/68304/diff/3/ > > > Testing > ------- > > make check > > > Thanks, > > Meng Zhu > >