aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Zameer Manji" <zma...@twopensource.com>
Subject Re: Review Request 27828: Move read-modify-write logic out of AttributeStore.Mutable for simpler layering.
Date Mon, 10 Nov 2014 21:13:10 GMT

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

Ship it!


Ship It!

- Zameer Manji


On Nov. 10, 2014, 12:50 p.m., Bill Farner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27828/
> -----------------------------------------------------------
> 
> (Updated Nov. 10, 2014, 12:50 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Zameer Manji.
> 
> 
> Bugs: AURORA-913
>     https://issues.apache.org/jira/browse/AURORA-913
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Simplify `AttributeStore.Mutable` by removing `setMaintenanceMode`, and remove the silent
merging behavior performed internally.
> 
> This change sets the stage for fixing AURORA-913, where i will introduce a thin layer
over `HostAttributeStore.Mutable` for the purposes of advertising when attributes change.
> 
> 
> Diffs
> -----
> 
>   src/main/java/org/apache/aurora/scheduler/http/Maintenance.java a7a807dd87bf695cb744891256950c4d4840969a

>   src/main/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImpl.java 2d382f752fb8d5997ad1f72100a8e032b7625656

>   src/main/java/org/apache/aurora/scheduler/state/MaintenanceController.java 21cfebdf11e0652e192cf08e35c8581b1246f7b5

>   src/main/java/org/apache/aurora/scheduler/storage/AttributeStore.java 8ca0dfdf07852c41dfcf43fad298780d0a45153b

>   src/main/java/org/apache/aurora/scheduler/storage/db/DbAttributeStore.java c8b3095ef44b3221e8c58f2b2ecd58e3175f13b4

>   src/main/java/org/apache/aurora/scheduler/storage/log/WriteAheadStorage.java 094d1c66e1f464687e0416bc652dba4b1bda0f7e

>   src/test/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImplTest.java af15c9543a9be4ad9916423ae650114d2b33aec2

>   src/test/java/org/apache/aurora/scheduler/state/MaintenanceControllerImplTest.java
4d96761e04a342ad3564bdba4afdc889f27ac123 
>   src/test/java/org/apache/aurora/scheduler/storage/db/DbAttributeStoreTest.java c0a1aa5096b507b440d4fe55d68292331c3cd903

>   src/test/java/org/apache/aurora/scheduler/storage/log/LogStorageTest.java 55a47844aea4c7ced5d7053a62bef29dc97698da

>   src/test/java/org/apache/aurora/scheduler/storage/log/SnapshotStoreImplTest.java ab0d501aac0c187d8fb9791ad43ef586b3aed5e7

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


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