----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30285/#review69682 -----------------------------------------------------------
Ship it! Ship It! - David McLaughlin On Jan. 26, 2015, 9:05 p.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30285/ > ----------------------------------------------------------- > > (Updated Jan. 26, 2015, 9:05 p.m.) > > > 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 > >