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

2016-04-09 Thread 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

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

2016-04-09 Thread John Sirois
> On April 7, 2016, 6:42 p.m., Zameer Manji wrote: > > src/main/java/org/apache/aurora/scheduler/discovery/CuratorServiceGroupMonitor.java, > > line 108 > > > > > > Shouldn't this be fatal? To me to looks like this

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

2016-04-09 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45902/#review128016 --- Master (0dd096d) is red with this patch.