Re: Review Request 16995: Only make the SchedulerDriver available after registered() has been called.

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

Re: Review Request 16995: Only make the SchedulerDriver available after registered() has been called.

2014-01-20 Thread Bill Farner
> On Jan. 16, 2014, 11:41 p.m., Kevin Sweeney wrote: > > src/main/java/org/apache/aurora/scheduler/Driver.java, lines 148-149 > > > > > > There's a (very small window) check-then-act race here. Should this be > > sync

Review Request 17133: Improve test coverage for TaskAssignerImpl.

2014-01-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17133/ --- Review request for Aurora, Kevin Sweeney and Suman Karumuri. Bugs: AURORA-63

Review Request 17131: Improve test coverage for CronJobManager.

2014-01-20 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17131/ --- Review request for Aurora, Suman Karumuri and Maxim Khutornenko. Bugs: AURORA-6