----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28914/#review64623 -----------------------------------------------------------
Ship it! Master (2aac148) is green with this patch. ./build-support/jenkins/build.sh I will refresh this build result if you post a review containing "@ReviewBot retry" - Aurora ReviewBot On Dec. 10, 2014, 8:06 p.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28914/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2014, 8:06 p.m.) > > > Review request for Aurora, Kevin Sweeney and Bill Farner. > > > Bugs: AURORA-967 > https://issues.apache.org/jira/browse/AURORA-967 > > > Repository: aurora > > > Description > ------- > > Adding PMD rule to check @Timed annotation placement. > > > Diffs > ----- > > config/pmd/custom.xml PRE-CREATION > > Diff: https://reviews.apache.org/r/28914/diff/ > > > Testing > ------- > > ``` > $ ./gradlew -Pq --rerun-tasks pmdMain > :buildSrc:compileJava UP-TO-DATE > :buildSrc:compileGroovy > :buildSrc:processResources UP-TO-DATE > :buildSrc:classes > :buildSrc:jar > :buildSrc:assemble > :buildSrc:compileTestJava UP-TO-DATE > :buildSrc:compileTestGroovy UP-TO-DATE > :buildSrc:processTestResources UP-TO-DATE > :buildSrc:testClasses UP-TO-DATE > :buildSrc:test UP-TO-DATE > :buildSrc:check UP-TO-DATE > :buildSrc:build > :pmdMain > /Users/mkhutornenko/workspace3/aurora/src/main/java/org/apache/aurora/scheduler/async/TaskScheduler.java:204: > A method must be overridable to have a @Timed annotation. > :pmdMain FAILED > > FAILURE: Build failed with an exception. > > * What went wrong: > Execution failed for task ':pmdMain'. > > 1 PMD rule violations were found. See the report at: > > file:///Users/mkhutornenko/workspace3/aurora/dist/reports/pmd/main.html > > * 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: 12.981 secs > ``` > > > Thanks, > > Maxim Khutornenko > >