Re: Review Request 41717: Very Very WIP: Add jittering as an option to BackoffStrategy.

2016-01-03 Thread Tony Dong
> On Dec. 27, 2015, 10:11 p.m., Stephan Erb wrote: > > commons/src/main/java/org/apache/aurora/common/util/Random.java, line 62 > > > > > > Having this implementation in Aurora doesn't feel right. It is rather > >

Re: Review Request 41717: Very Very WIP: Add jittering as an option to BackoffStrategy.

2016-01-01 Thread Tony Dong
> On Dec. 27, 2015, 10:11 p.m., Stephan Erb wrote: > > Thanks for your effort. > > > > I believe `ExpBackoffEqualJitter.java` is the most usefull strategy. It is > > the closest to the current design while still resolving the problem > > depicted in the AWS article > >

Re: Review Request 41717: Very Very WIP: Add jittering as an option to BackoffStrategy.

2016-01-01 Thread Tony Dong
> On Dec. 27, 2015, 10:11 p.m., Stephan Erb wrote: > > commons/src/main/java/org/apache/aurora/common/util/Random.java, line 62 > > > > > > Having this implementation in Aurora doesn't feel right. It is rather > >

Re: Review Request 41717: Very Very WIP: Add jittering as an option to BackoffStrategy.

2016-01-01 Thread Stephan Erb
> On Dec. 27, 2015, 11:11 p.m., Stephan Erb wrote: > > Thanks for your effort. > > > > I believe `ExpBackoffEqualJitter.java` is the most usefull strategy. It is > > the closest to the current design while still resolving the problem > > depicted in the AWS article > >

Re: Review Request 41717: Very Very WIP: Add jittering as an option to BackoffStrategy.

2015-12-27 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41717/#review111942 --- Thanks for your effort. I believe `ExpBackoffEqualJitter.java` is

Re: Review Request 41717: Very Very WIP: Add jittering as an option to BackoffStrategy.

2015-12-27 Thread Bill Farner
> On Dec. 27, 2015, 2:11 p.m., Stephan Erb wrote: > > Thanks for your effort. > > > > I believe `ExpBackoffEqualJitter.java` is the most usefull strategy. It is > > the closest to the current design while still resolving the problem > > depicted in the AWS article > >

Re: Review Request 41717: Very Very WIP: Add jittering as an option to BackoffStrategy.

2015-12-25 Thread Tony Dong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41717/#review111889 --- I'm putting up this WIP review to get some general feedback. I