Re: Review Request 41777: Use slf4j throughout the scheduler.

2016-01-03 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41777/#review112479 --- Ship it! - Zameer Manji On Dec. 29, 2015, 4:06 p.m., Bill

Re: Review Request 41762: Upgrade ZooKeeper dep to 3.4.2

2016-01-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41762/ --- (Updated Jan. 3, 2016, 8:55 p.m.) Review request for Aurora, John Sirois and

Re: Review Request 41785: Remove scheduler log scaffolding

2016-01-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41785/ --- (Updated Jan. 3, 2016, 9:04 p.m.) Review request for Aurora, John Sirois and

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: Add jittering to TruncatedBinaryBackoff.

2016-01-03 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41717/#review112459 --- Master (4a9f2ff) is green with this patch.

Re: Review Request 41717: Add jittering to TruncatedBinaryBackoff.

2016-01-03 Thread Tony Dong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41717/ --- (Updated Jan. 3, 2016, 3:24 p.m.) Review request for Aurora, Stephan Erb and

Re: Review Request 41846: Add a list of Aurora design documents

2016-01-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41846/#review112465 --- Ship it! Ship It! - Bill Farner On Jan. 2, 2016, 4:43 a.m.,

Re: Review Request 41786: Remove several scheduler command line arguments.

2016-01-03 Thread Bill Farner
> On Dec. 31, 2015, 6:41 a.m., Stephan Erb wrote: > > src/main/java/org/apache/aurora/scheduler/thrift/aop/AopModule.java, line 51 > > > > > > I've always thought those are somewhat meant to be used for backup > >

Re: Review Request 41717: Add jittering to TruncatedBinaryBackoff.

2016-01-03 Thread Bill Farner
--- 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.

Re: Review Request 41717: Add jittering to TruncatedBinaryBackoff.

2016-01-03 Thread Tony Dong
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41717/ --- (Updated Jan. 3, 2016, 5:51 p.m.) Review request for Aurora, Stephan Erb and

Re: Review Request 41717: Add jittering to TruncatedBinaryBackoff.

2016-01-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41717/#review112470 --- Doh! I'm ready to apply this, but i can't land the patch and give

Re: Review Request 41717: Add jittering to TruncatedBinaryBackoff.

2016-01-03 Thread Tony Dong
> On Jan. 3, 2016, 5:55 p.m., Bill Farner wrote: > > Doh! I'm ready to apply this, but i can't land the patch and give you > > credit on the commit log without access to your e-mail in ReviewBoard. > > > > Please go here: https://reviews.apache.org/account/preferences/#profile > > and untick

Re: Review Request 41844: Update and slightly extend the beginner tutorial

2016-01-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41844/#review112464 --- This is great, thanks for the cleanup! Can you rebase your branch

Re: Review Request 41853: Add recent Aurora-related talks

2016-01-03 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41853/#review112468 --- Ship it! Great stuff, thanks!! - Bill Farner On Jan. 2, 2016,