----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28914/#review64624 -----------------------------------------------------------
config/pmd/custom.xml <https://reviews.apache.org/r/28914/#comment107411> Remove this comment. config/pmd/custom.xml <https://reviews.apache.org/r/28914/#comment107409> Name = Aurora? config/pmd/custom.xml <https://reviews.apache.org/r/28914/#comment107410> Used with the @Timed annotation. Also consider adding a link to the guice docs on method interceptor limitations for those unfamiliar. config/pmd/custom.xml <https://reviews.apache.org/r/28914/#comment107412> My XPath-foo is lacking - will this catch all of the following forms: ```java @Timed("test_var") private void myMethod() {} @Timed void myMethod2() {} @Timed(value = "time_var") static void myMethod3() {} ``` Also is there a way to limit this match to the Timed annotation from com.twitter.common.inject.TimedInterceptor.Timed? - Kevin Sweeney On Dec. 10, 2014, 12: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, 12: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 > >