aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aurora ReviewBot" <wfar...@apache.org>
Subject Re: Review Request 36710: Add an executor service decorator that gates async operations.
Date Thu, 23 Jul 2015 17:33:50 GMT

-----------------------------------------------------------
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-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot retry"

- Aurora ReviewBot


On July 23, 2015, 5:20 p.m., Bill Farner wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> Bugs: AURORA-1395
>     https://issues.apache.org/jira/browse/AURORA-1395
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> In the current state of the scheduler, we start async work in the context of a transaction.
 As we have observed in the linked ticket, this work can read data that is inconsistent with
the transaction context if it starts before the transaction completes.  The purpose of this
utility is to let async work queue until the originating transaction completes.
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/aurora/scheduler/async/DelayExecutor.java PRE-CREATION 
>   src/main/java/org/apache/aurora/scheduler/async/FlushableWorkQueue.java PRE-CREATION

>   src/main/java/org/apache/aurora/scheduler/async/GatedDelayExecutor.java PRE-CREATION

>   src/main/java/org/apache/aurora/scheduler/events/PubsubEventModule.java c85979dedd0ef2c515453a33c9a36d52865eb548

>   src/test/java/org/apache/aurora/scheduler/async/GatedDelayExecutorTest.java PRE-CREATION

> 
> Diff: https://reviews.apache.org/r/36710/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Bill Farner
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message