Re: Review Request 36710: Add an executor service decorator that gates async operations.

2015-07-24 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36710/#review93000 --- Ship it! Ship It! - Kevin Sweeney On July 23, 2015, 10:20 a.m.,

Re: Review Request 36710: Add an executor service decorator that gates async operations.

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

Re: Review Request 36710: Add an executor service decorator that gates async operations.

2015-07-23 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36710/ --- (Updated July 23, 2015, 5:20 p.m.) Review request for Aurora and Kevin Sweeney.

Re: Review Request 36710: Add an executor service decorator that gates async operations.

2015-07-23 Thread Bill Farner
> On July 23, 2015, 12:11 a.m., Kevin Sweeney wrote: > > src/main/java/org/apache/aurora/scheduler/async/GatedScheduledExecutorService.java, > > line 36 > > > > > > Consider extending > > http://docs.guava-libraries

Re: Review Request 36710: Add an executor service decorator that gates async operations.

2015-07-22 Thread Kevin Sweeney
> On July 22, 2015, 5:11 p.m., Kevin Sweeney wrote: > > src/main/java/org/apache/aurora/scheduler/async/GatedScheduledExecutorService.java, > > line 36 > > > > > > Consider extending > > http://docs.guava-libraries.

Re: Review Request 36710: Add an executor service decorator that gates async operations.

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

Re: Review Request 36710: Add an executor service decorator that gates async operations.

2015-07-22 Thread Bill Farner
> On July 23, 2015, 12:11 a.m., Kevin Sweeney wrote: > > src/main/java/org/apache/aurora/scheduler/async/GatedScheduledExecutorService.java, > > line 36 > > > > > > Consider extending > > http://docs.guava-libraries

Re: Review Request 36710: Add an executor service decorator that gates async operations.

2015-07-22 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36710/#review92685 --- src/main/java/org/apache/aurora/scheduler/async/GatedScheduledExecu

Review Request 36710: Add an executor service decorator that gates async operations.

2015-07-22 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36710/ --- Review request for Aurora, Kevin Sweeney and Zameer Manji. Bugs: AURORA-1395