Re: Review Request 17785: Ignore THROTTLED tasks when looking for tasks associated with a slave.

2014-02-06 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17785/#review33877 --- Rebased, re-run tests and pushed to master. - Maxim Khutornenko O

Re: Review Request 17785: Ignore THROTTLED tasks when looking for tasks associated with a slave.

2014-02-06 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17785/#review33873 --- Ship it! LGTM mod potential need to rebase and run tests again. -

Re: Review Request 17785: Ignore THROTTLED tasks when looking for tasks associated with a slave.

2014-02-06 Thread Kevin Sweeney
> On Feb. 6, 2014, 4:05 p.m., Kevin Sweeney wrote: > > src/main/thrift/org/apache/aurora/gen/api.thrift, line 273 > > > > > > I'm not clear on why THROTTLED is an active state - doesn't it precede > > PENDING? > > Ma

Re: Review Request 17785: Ignore THROTTLED tasks when looking for tasks associated with a slave.

2014-02-06 Thread Maxim Khutornenko
> On Feb. 7, 2014, 12:05 a.m., Kevin Sweeney wrote: > > src/main/thrift/org/apache/aurora/gen/api.thrift, line 273 > > > > > > I'm not clear on why THROTTLED is an active state - doesn't it precede > > PENDING? THROT

Re: Review Request 17785: Ignore THROTTLED tasks when looking for tasks associated with a slave.

2014-02-06 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17785/#review33868 --- src/main/thrift/org/apache/aurora/gen/api.thrift

Re: Review Request 17785: Ignore THROTTLED tasks when looking for tasks associated with a slave.

2014-02-06 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17785/#review33802 --- Ship it! src/main/java/org/apache/aurora/scheduler/async/TaskSched

Review Request 17785: Ignore THROTTLED tasks when looking for tasks associated with a slave.

2014-02-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17785/ --- Review request for Aurora, Kevin Sweeney and Maxim Khutornenko. Bugs: AURORA-19