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

2014-01-21 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16995/#review32389 --- Ship it! Ship It! - Kevin Sweeney On Jan. 20, 2014, 2:24 p.m., B

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

2014-01-21 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16995/#review32372 --- Ship it! Ship It! - Suman Karumuri On Jan. 20, 2014, 10:24 p.m.,

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

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

2014-01-16 Thread Suman Karumuri
> 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

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

2014-01-16 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16995/#review32096 --- src/main/java/org/apache/aurora/scheduler/Driver.java

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

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