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

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

2015-12-31 Thread Stephan Erb
> On Dez. 31, 2015, 3:41 nachm., Stephan Erb wrote: > > src/main/java/org/apache/aurora/scheduler/storage/log/LogStorageModule.java, > > line 67 > > > > > > These docs are now out of date and can probably be dropped:

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

2015-12-31 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41786/#review112401 --- Ship it! Ship It! - Stephan Erb On Dez. 30, 2015, 2:52 vorm., B

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

2015-12-31 Thread Bill Farner
> On Dec. 31, 2015, 6:41 a.m., Stephan Erb wrote: > > src/main/java/org/apache/aurora/scheduler/storage/log/LogStorageModule.java, > > line 67 > > > > > > These docs are now out of date and can probably be dropped:

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

2015-12-31 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41786/#review112392 --- src/main/java/org/apache/aurora/scheduler/storage/log/LogStorageMo

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

2015-12-30 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41786/#review112362 --- Ship it! Ship It! - Zameer Manji On Dec. 29, 2015, 5:52 p.m., B

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

2015-12-29 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41786/#review112279 --- Ship it! Master (80ae5e8) is green with this patch. ./build-supp

Review Request 41786: Remove several scheduler command line arguments.

2015-12-29 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41786/ --- Review request for Aurora, Joshua Cohen and Zameer Manji. Repository: aurora