Re: Review Request 16740: When rescheduling a task, send it to the THROTTLED state if it has been penalized for flapping.

2014-01-14 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16740/#review31785 --- Ship it! Ship It! - Suman Karumuri On Jan. 13, 2014, 11:54 p.m.,

Re: Review Request 16740: When rescheduling a task, send it to the THROTTLED state if it has been penalized for flapping.

2014-01-13 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16740/ --- (Updated Jan. 13, 2014, 11:54 p.m.) Review request for Aurora, Kevin Sweeney, S

Re: Review Request 16740: When rescheduling a task, send it to the THROTTLED state if it has been penalized for flapping.

2014-01-13 Thread Bill Farner
> On Jan. 13, 2014, 11:42 p.m., Kevin Sweeney wrote: > > src/main/java/org/apache/aurora/scheduler/async/AsyncModule.java, line 237 > > > > > > Can just call install(...) iirc. Hah! Thanks for catching, looks like th

Re: Review Request 16740: When rescheduling a task, send it to the THROTTLED state if it has been penalized for flapping.

2014-01-13 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16740/#review31419 --- Ship it! Shipit was lost in the mail... src/main/java/org/apache/

Re: Review Request 16740: When rescheduling a task, send it to the THROTTLED state if it has been penalized for flapping.

2014-01-13 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16740/#review31667 --- Post-weekend ping. Waiting on Kevin and Suman here. - Bill Farner

Re: Review Request 16740: When rescheduling a task, send it to the THROTTLED state if it has been penalized for flapping.

2014-01-09 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16740/#review31484 --- Ship it! Ship It! - Maxim Khutornenko On Jan. 9, 2014, 10:53 p.m

Re: Review Request 16740: When rescheduling a task, send it to the THROTTLED state if it has been penalized for flapping.

2014-01-09 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16740/ --- (Updated Jan. 9, 2014, 10:53 p.m.) Review request for Aurora, Kevin Sweeney, Su

Re: Review Request 16740: When rescheduling a task, send it to the THROTTLED state if it has been penalized for flapping.

2014-01-09 Thread Bill Farner
> On Jan. 9, 2014, 2:25 a.m., Maxim Khutornenko wrote: > > Would it make sense to keep the StorageBackfillTest class as a placeholder > > for future schema migration tests? Will come handy next time we go through > > a schema migration. I'm generally pretty firmly against keeping dead code aro

Re: Review Request 16740: When rescheduling a task, send it to the THROTTLED state if it has been penalized for flapping.

2014-01-08 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16740/#review31421 --- Would it make sense to keep the StorageBackfillTest class as a place

Review Request 16740: When rescheduling a task, send it to the THROTTLED state if it has been penalized for flapping.

2014-01-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16740/ --- Review request for Aurora, Kevin Sweeney, Suman Karumuri, and Maxim Khutornenko.