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

(Updated April 7, 2016, 5:07 p.m.)


Review request for Aurora, Bill Farner and Zameer Manji.


Changes
-------

Document `CuratorServiceGroupMonitor`.

Also tighten the contract of the `memberSelector` `Predicate`, passing
it just the node name and not the full path.

 
src/main/java/org/apache/aurora/scheduler/discovery/CuratorServiceGroupMonitor.java
     | 20 +++++++++++++++++++-
 
src/test/java/org/apache/aurora/scheduler/discovery/CuratorServiceGroupMonitorTest.java
 |  2 +-
 2 files changed, 20 insertions(+), 2 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 ad5ec5c4afe4dd5c2189d0680692a7409f0417a9 
  
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

Reply via email to