Re: Review Request 33105: Return Iterable from TaskStore.fetchTasks to allow for streaming.

2015-04-14 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33105/#review80123 --- Ship it! Master (1dc11fb) is green with this patch.

Re: Review Request 33105: Return Iterable from TaskStore.fetchTasks to allow for streaming.

2015-04-13 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33105/#review79960 --- Ship it! Ship It! - Maxim Khutornenko On April 11, 2015, 6:22

Re: Review Request 33105: Return Iterable from TaskStore.fetchTasks to allow for streaming.

2015-04-13 Thread Bill Farner
On April 13, 2015, 11:06 p.m., Kevin Sweeney wrote: src/main/java/org/apache/aurora/scheduler/async/RescheduleCalculator.java, lines 149-152 https://reviews.apache.org/r/33105/diff/1/?file=924537#file924537line149 Inline this as FluentIterable.from(...).first() `first()` changes

Re: Review Request 33105: Return Iterable from TaskStore.fetchTasks to allow for streaming.

2015-04-11 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33105/#review79809 --- Master (eb9e408) is red with this patch.