----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41717/#review112469 -----------------------------------------------------------
Ship it! LGTM after Stephan's comment is addressed. commons/src/main/java/org/apache/aurora/common/util/TruncatedBinaryBackoff.java (line 53) <https://reviews.apache.org/r/41717/#comment172922> convention nit: `checkNotNull(random)` - Bill Farner On Jan. 3, 2016, 7:39 a.m., Tony Dong wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41717/ > ----------------------------------------------------------- > > (Updated Jan. 3, 2016, 7:39 a.m.) > > > Review request for Aurora, Stephan Erb and Bill Farner. > > > Repository: aurora > > > Description > ------- > > Add jittering to TruncatedBinaryBackoff. > > > Diffs > ----- > > commons/src/main/java/org/apache/aurora/common/args/parsers/EnumParser.java > 9f6a3ff977ef6a0e2d472a8e2cf90e7f9a63a985 > > commons/src/main/java/org/apache/aurora/common/util/TruncatedBinaryBackoff.java > fd74b9f37c6cc24c7ea1cb239ba6354661d931e2 > > commons/src/test/java/org/apache/aurora/common/util/TruncatedBinaryBackoffTest.java > 127a60331f8560d98733bd16654087f840abef72 > src/main/java/org/apache/aurora/scheduler/offers/OffersModule.java > fbc589e9a7592cce6d92c4e987cde2e056406c3a > > src/main/java/org/apache/aurora/scheduler/scheduling/RescheduleCalculator.java > 291bf5f0baefef6dd10d19ec89e173ce495e6380 > > Diff: https://reviews.apache.org/r/41717/diff/ > > > Testing > ------- > > > Thanks, > > Tony Dong > >
