----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30285/ -----------------------------------------------------------
Review request for Aurora and Zameer Manji. Repository: aurora Description ------- This broke as a result of me reverting a change done in https://reviews.apache.org/r/28920/ when resolving merge conflicts. Prior to that patch, a call to `LOG.fine` was guarded by an if statement, which was removed. I reverted that since the purpose of the guard was performance-related (protobuf's toString() function can be very expensive. For some unknown reason my local builds continued to pass after this change, despite failing in jenkins. This was made worse by https://issues.apache.org/jira/browse/AURORA-1060, which caused review bot to fail at a point before this check was done. Diffs ----- config/legacy_untested_classes.txt 3429e35376cf71863dcc63c9c667aa70b0ec6b22 Diff: https://reviews.apache.org/r/30285/diff/ Testing ------- Thanks, Bill Farner
