----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26298/#review55295 -----------------------------------------------------------
src/main/python/apache/aurora/client/api/scheduler_client.py <https://reviews.apache.org/r/26298/#comment95682> Why not just add a break into the Exception catcher instead? - Maxim Khutornenko On Oct. 3, 2014, 12:18 a.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26298/ > ----------------------------------------------------------- > > (Updated Oct. 3, 2014, 12:18 a.m.) > > > Review request for Aurora, Mark Chu-Carroll and Maxim Khutornenko. > > > Repository: aurora > > > Description > ------- > > A few times when making changes, i've found myself confused at a stalled test > and spiked CPU, only to find that my test should have failed, but an > exception is trapped in this retry loop. The key change here is that unknown > exceptions will break the loop. > > Making this change pointed out what should have been a test failure in > `test_transient_error`, where an exception caused by an unexpected call to > `getVersion` was swallowed. > > > Diffs > ----- > > src/main/python/apache/aurora/client/api/scheduler_client.py > b400cb2dbdb35077fc2c4a6e161c2959a9217317 > src/test/python/apache/aurora/client/api/test_scheduler_client.py > 1cbfbf86e903d890baac7d34461109f9beaff442 > > Diff: https://reviews.apache.org/r/26298/diff/ > > > Testing > ------- > > ./pants src/test/python:all -vxs > > > Thanks, > > Bill Farner > >