aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Farner" <wfar...@apache.org>
Subject Review Request 38336: Make async work queue gating thread-local.
Date Sun, 13 Sep 2015 17:02:39 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38336/
-----------------------------------------------------------

Review request for Aurora and Maxim Khutornenko.


Bugs: AURORA-1459
    https://issues.apache.org/jira/browse/AURORA-1459


Repository: aurora


Description
-------

This effectively makes `GatedDelayExecutor` (renamed to `GatingDelayExecutor`) transaction-aware,
and avoids unnecessarily delaying independent async work that is not part of a transaction.


Diffs
-----

  config/findbugs/excludeFilter.xml 7c653022fdc1a33e71da08c2adeaa5760153a08b 
  src/main/java/org/apache/aurora/scheduler/async/AsyncModule.java 217b9c0347b259e76180a7021c9c0f9c0c34c89c

  src/main/java/org/apache/aurora/scheduler/async/FlushableWorkQueue.java 11a1c2a7a920eefeb92cbf230866ed107941f642

  src/main/java/org/apache/aurora/scheduler/async/GatedDelayExecutor.java 9d4cfcf1f6b9da4261e2b0252d39aeeb1b4427bb

  src/main/java/org/apache/aurora/scheduler/storage/db/DbModule.java 6da6193c47580b3e4446371f0df798eccc24f809

  src/main/java/org/apache/aurora/scheduler/storage/db/DbStorage.java 60365709ced5314e3563ab5d75d7a0fffda9ac91

  src/test/java/org/apache/aurora/scheduler/app/SchedulerIT.java 49411289299bfc230e8c6e0ce6fa96e619f57524

  src/test/java/org/apache/aurora/scheduler/async/GatedDelayExecutorTest.java 28676332f74c6a4cffeb867ef5612e12d629b5b8

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

  src/test/java/org/apache/aurora/scheduler/pruning/TaskHistoryPrunerTest.java 0c7da074515e1fad534617f9b8f73d939caf948b

  src/test/java/org/apache/aurora/scheduler/storage/db/DbStorageTest.java 6dd50264b7d119aa5b5f96ab153794e95c443822


Diff: https://reviews.apache.org/r/38336/diff/


Testing
-------


Thanks,

Bill Farner


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