----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32990/#review79556 -----------------------------------------------------------
Master (7e3e7c9) is red with this patch. ./build-support/jenkins/build.sh make[1]: Leaving directory `/home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/thrift/thrift-0.9.1' make: Leaving directory `/home/jenkins/jenkins-slave/workspace/AuroraBot/build-support/thrift' :api:classesThriftNote: Some input files use unchecked or unsafe operations. Note: Recompile with -Xlint:unchecked for details. :api:checkPython :api:generateThriftEntitiesJava :api:classesThriftEntities :api:compileJava UP-TO-DATE :api:generateThriftResources :api:processResources UP-TO-DATE :api:classes :api:jar :compileJavaNote: Writing file:/home/jenkins/jenkins-slave/workspace/AuroraBot/dist/classes/main/com/twitter/common/args/apt/cmdline.arg.info.txt.2 :processResources :classes :jar :startScripts :distTar :distZip :assemble :compileJmhJava :processJmhResources UP-TO-DATE :jmhClasses :checkstyleJmh[ant:checkstyle] /home/jenkins/jenkins-slave/workspace/AuroraBot/src/jmh/java/org/apache/aurora/benchmark/UpdateStoreBenchmarks.java:67: Wrong order for 'com.twitter.common.inject.Bindings' import. Order should be: java, javax, scala, com, net, org. Each group should be separated by a single blank line. FAILED FAILURE: Build failed with an exception. * What went wrong: Execution failed for task ':checkstyleJmh'. > Checkstyle rule violations were found. See the report at: > file:///home/jenkins/jenkins-slave/workspace/AuroraBot/dist/reports/checkstyle/jmh.xml * Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. BUILD FAILED Total time: 1 mins 39.707 secs I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On April 9, 2015, 6:43 p.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32990/ > ----------------------------------------------------------- > > (Updated April 9, 2015, 6:43 p.m.) > > > Review request for Aurora and Bill Farner. > > > Bugs: AURORA-1263 > https://issues.apache.org/jira/browse/AURORA-1263 > > > Repository: aurora > > > Description > ------- > > Using event collections with "select" avoids unnecessary row explosion and > provdes subsecond execution time for a large job update that used to take > half a minute. > > Before: > ``` > Benchmark (instances) Mode Cnt > Score Error Units > UpdateStoreBenchmarks.JobDetailsBenchmark.run 1000 thrpt 5 > 15.595 ± 1.056 ops/s > UpdateStoreBenchmarks.JobDetailsBenchmark.run 5000 thrpt 5 > 3.143 ± 0.259 ops/s > UpdateStoreBenchmarks.JobDetailsBenchmark.run 10000 thrpt 5 > 1.539 ± 0.075 ops/s > ``` > > After: > ``` > Benchmark (instances) Mode Cnt > Score Error Units > UpdateStoreBenchmarks.JobDetailsBenchmark.run 1000 thrpt 5 > 278.405 ± 57.311 ops/s > UpdateStoreBenchmarks.JobDetailsBenchmark.run 5000 thrpt 5 > 65.386 ± 6.455 ops/s > UpdateStoreBenchmarks.JobDetailsBenchmark.run 10000 thrpt 5 > 33.353 ± 1.974 ops/s > ``` > > > Diffs > ----- > > src/jmh/java/org/apache/aurora/benchmark/UpdateStoreBenchmarks.java > PRE-CREATION > > src/main/resources/org/apache/aurora/scheduler/storage/db/JobUpdateDetailsMapper.xml > 4536cd35ffc231a5d2f6c1b5bf2aaa084ce3ad1d > > Diff: https://reviews.apache.org/r/32990/diff/ > > > Testing > ------- > > ./gradlew -Pq build > Manual testing in vagrant. > > > Thanks, > > Maxim Khutornenko > >