Re: Review Request 24206: Add an UpdateStrategy abstraction, with two implementations.

2014-08-04 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24206/#review49466 --- Ship it! Ship It!

Re: Review Request 24206: Add an UpdateStrategy abstraction, with two implementations.

2014-08-04 Thread Bill Farner
On Aug. 4, 2014, 5:34 p.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/updater/strategy/ActiveLimitedStrategy.java, line 36 https://reviews.apache.org/r/24206/diff/1/?file=649183#file649183line36 protected? Done. On Aug. 4, 2014, 5:34 p.m., Kevin Sweeney

Re: Review Request 24206: Add an UpdateStrategy abstraction, with two implementations.

2014-08-03 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24206/#review49435 --- Ship it!