Re: Review Request 54299: Extend warm-up time by `max_consecutive_failures` attempts.

2016-12-04 Thread Santhosh Kumar Shanmugham
> On Dec. 2, 2016, 11:54 a.m., Zameer Manji wrote: > > It took me a long time to understand this after staring at the tests, but I > > think this is correct. > > > > This is unfortunately a little complex to understand. For bonus points, > > would it be possible to encode some of this informat

Re: Review Request 54299: Extend warm-up time by `max_consecutive_failures` attempts.

2016-12-04 Thread Santhosh Kumar Shanmugham
> On Dec. 2, 2016, 1:44 p.m., Joshua Cohen wrote: > > src/main/python/apache/aurora/executor/common/health_checker.py, lines > > 115-117 > > > > > > There still exists the chance for a backwards incompatibility here