Re: Review Request 37001: Centralizing offer/task matching in TaskAssigner.

2015-08-03 Thread Maxim Khutornenko
> On Aug. 3, 2015, 5:38 p.m., Zameer Manji wrote: > > src/main/java/org/apache/aurora/scheduler/scheduling/TaskScheduler.java, > > line 111 > > > > > > nit: Was it needed to flip the BiCache type here? If not, it wo

Re: Review Request 37001: Centralizing offer/task matching in TaskAssigner.

2015-08-03 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37001/#review93922 --- Ship it! src/main/java/org/apache/aurora/scheduler/scheduling/Task

Re: Review Request 37001: Centralizing offer/task matching in TaskAssigner.

2015-07-31 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37001/#review93817 --- Ship it! - Bill Farner On July 31, 2015, 11:03 p.m., Maxim Khutor

Re: Review Request 37001: Centralizing offer/task matching in TaskAssigner.

2015-07-31 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37001/#review93816 --- Ship it! Master (a45a8ee) is green with this patch. ./build-suppo