Re: Review Request 16743: Use a separate thread for Driver.join() to avoid stalling the executor service.

2014-01-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16743/#review31416 --- Thanks, folks. This is now on master. - Bill Farner On Jan. 8, 2

Re: Review Request 16743: Use a separate thread for Driver.join() to avoid stalling the executor service.

2014-01-08 Thread Mark Chu-Carroll
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16743/#review31415 --- Ship it! Ship It! - Mark Chu-Carroll On Jan. 8, 2014, 6:15 p.m.,

Re: Review Request 16743: Use a separate thread for Driver.join() to avoid stalling the executor service.

2014-01-08 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16743/#review31413 --- Ship it! Ship It! - Maxim Khutornenko On Jan. 8, 2014, 11:15 p.m

Re: Review Request 16743: Use a separate thread for Driver.join() to avoid stalling the executor service.

2014-01-08 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16743/#review31412 --- Ship it! Ship It! - Kevin Sweeney On Jan. 8, 2014, 3:15 p.m., Bi

Re: Review Request 16743: Use a separate thread for Driver.join() to avoid stalling the executor service.

2014-01-08 Thread Bill Farner
> On Jan. 8, 2014, 11:20 p.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/SchedulerLifecycle.java, line 158 > > > > > > How would the unhandled exception surface from here? Does it make sens

Re: Review Request 16743: Use a separate thread for Driver.join() to avoid stalling the executor service.

2014-01-08 Thread Kevin Sweeney
> On Jan. 8, 2014, 3:20 p.m., Maxim Khutornenko wrote: > > src/main/java/org/apache/aurora/scheduler/SchedulerLifecycle.java, line 158 > > > > > > How would the unhandled exception surface from here? Does it make sense

Re: Review Request 16743: Use a separate thread for Driver.join() to avoid stalling the executor service.

2014-01-08 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16743/#review31407 --- src/main/java/org/apache/aurora/scheduler/SchedulerLifecycle.java <

Review Request 16743: Use a separate thread for Driver.join() to avoid stalling the executor service.

2014-01-08 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16743/ --- Review request for Aurora, Mark Chu-Carroll and Maxim Khutornenko. Bugs: AURORA