Re: Review Request 46603: Introduce command line option to control the offer filter duration

2016-04-27 Thread Stephan Erb
> On April 24, 2016, 5:48 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/offers/OffersModule.java, line 51 > > > > > > Does this default value effect the same behavior as before the patch? >

Re: Review Request 46603: Introduce command line option to control the offer filter duration

2016-04-27 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46603/ --- (Updated April 28, 2016, 12:12 a.m.) Review request for Aurora, Maxim

Re: Review Request 46603: Introduce command line option to control the offer filter duration

2016-04-27 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46603/#review130855 --- Ship it! Master (7e30ebe) is green with this patch.

Re: Review Request 46603: Introduce command line option to control the offer filter duration

2016-04-27 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46603/ --- (Updated April 27, 2016, 11:49 p.m.) Review request for Aurora, Maxim

Re: Review Request 46603: Introduce command line option to control the offer filter duration

2016-04-25 Thread Maxim Khutornenko
> On April 24, 2016, 3:48 p.m., Bill Farner wrote: > > src/main/java/org/apache/aurora/scheduler/offers/OffersModule.java, line 51 > > > > > > Does this default value effect the same behavior as before the patch? >

Re: Review Request 46603: Introduce command line option to control the offer filter duration

2016-04-24 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46603/#review130306 --- Fix it, then Ship it! LGTM overall, one question that i've

Re: Review Request 46603: Introduce command line option to control the offer filter duration

2016-04-23 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46603/#review130262 --- Ship it! Master (d339036) is green with this patch.

Re: Review Request 46603: Introduce command line option to control the offer filter duration

2016-04-23 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46603/#review130260 ---

Review Request 46603: Introduce command line option to control the offer filter duration

2016-04-23 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46603/ --- Review request for Aurora, Maxim Khutornenko and Bill Farner. Bugs: