----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42475/#review115203 -----------------------------------------------------------
Ship it! Ship It! - Maxim Khutornenko On Jan. 18, 2016, 10:37 p.m., John Sirois wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42475/ > ----------------------------------------------------------- > > (Updated Jan. 18, 2016, 10:37 p.m.) > > > Review request for Aurora, Maxim Khutornenko and Bill Farner. > > > Repository: aurora > > > Description > ------- > > The `TierConfig` binding fix for the `SchedulingBenchmarks` in > https://reviews.apache.org/r/42073 was also needed for the > `StatusUpdateBenchmarks` and a binding for `ConfigurationManager` was > missing for the `ThriftApiBenchmarks` as-of > https://reviews.apache.org/r/41711/. > > While debugging these failures I noticed a new version of jmh had been > released recently, so also upgraded to that. No changes of import, but > the changelog can be read here: > http://hg.openjdk.java.net/code-tools/jmh/ > > build.gradle | 4 ++-- > src/jmh/java/org/apache/aurora/benchmark/StatusUpdateBenchmark.java | 8 > ++++---- > src/jmh/java/org/apache/aurora/benchmark/ThriftApiBenchmarks.java | 6 > ++++++ > 3 files changed, 12 insertions(+), 6 deletions(-) > > > Diffs > ----- > > build.gradle 169fb2535c6faddd0546d12d070cbaab1830c832 > src/jmh/java/org/apache/aurora/benchmark/StatusUpdateBenchmark.java > 3898596f6a657a6908d4f6fb9fde952186827aa7 > src/jmh/java/org/apache/aurora/benchmark/ThriftApiBenchmarks.java > fd182aa60b90a26fc073cd5821c64b7129439741 > > Diff: https://reviews.apache.org/r/42475/diff/ > > > Testing > ------- > > Both of these now work on master: > `./gradlew jmh -Pbenchmarks='StatusUpdateBenchmark.*'` > `./gradlew jmh -Pbenchmarks='ThriftApiBenchmark.*'` > > I also checked that a full run of `./gradlew jmh` was now green for > all benchmarks. > > > Thanks, > > John Sirois > >
