Re: Review Request 45902: Introduce a Curator-based `ServiceGroupMonitor`.

[John Sirois]

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

(Updated April 9, 2016, 8:37 p.m.)


Review request for Aurora, Bill Farner and Zameer Manji.


Changes
-------

Cleanup CuratorServiceGroupMonitor and test.

 
src/main/java/org/apache/aurora/scheduler/discovery/CuratorServiceGroupMonitor.java
     |  8 +-------
 
src/test/java/org/apache/aurora/scheduler/discovery/CuratorServiceGroupMonitorTest.java
 | 24 +++++++++++-------------
 2 files changed, 12 insertions(+), 20 deletions(-)


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


Repository: aurora


Description
-------

build.gradle                                                                    
        |   5 ++
 
src/main/java/org/apache/aurora/scheduler/discovery/CuratorServiceGroupMonitor.java
     |  94 +++++++++++++++++++++++
 
src/test/java/org/apache/aurora/scheduler/discovery/CuratorServiceGroupMonitorTest.java
 | 205 ++++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 304 insertions(+)


Diffs (updated)
-----

  build.gradle fc61adfcbaf12c8ea45f71c7efe69d4fe39597c6 
  
src/main/java/org/apache/aurora/scheduler/discovery/CuratorServiceGroupMonitor.java
 PRE-CREATION 
  
src/test/java/org/apache/aurora/scheduler/discovery/CuratorServiceGroupMonitorTest.java
 PRE-CREATION 

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


Testing
-------

Locally green: `./gradlew -Pq build`.


Thanks,

John Sirois