----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28914/#review64616 -----------------------------------------------------------
Ship it! Ship It! - Kevin Sweeney On Dec. 10, 2014, 11:57 a.m., Maxim Khutornenko wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28914/ > ----------------------------------------------------------- > > (Updated Dec. 10, 2014, 11:57 a.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/design.xml 2741e407e35826e73bd4dc8335c0ef6294086398 > > 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 > >