You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tez.apache.org by "Jonathan Eagles (JIRA)" <ji...@apache.org> on 2014/03/13 17:41:45 UTC

[jira] [Commented] (TEZ-813) Remove unused imports across project

    [ https://issues.apache.org/jira/browse/TEZ-813?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13933510#comment-13933510 ] 

Jonathan Eagles commented on TEZ-813:
-------------------------------------

Updated the patch. I know it may not be a high priority, but I would prefer to fix these all at once rather that one checkin at a time.

> Remove unused imports across project
> ------------------------------------
>
>                 Key: TEZ-813
>                 URL: https://issues.apache.org/jira/browse/TEZ-813
>             Project: Apache Tez
>          Issue Type: Improvement
>    Affects Versions: 0.3.0
>            Reporter: Jonathan Eagles
>            Assignee: Jonathan Eagles
>              Labels: newbie
>         Attachments: TEZ-813.patch
>
>
> Noticed a few unused imports. pmd scan shows 63 imports like this. Fixing them all at one
> modified pom plugin management to add pmd configuration (for detection purposes)
> {code:xml}
>          <plugin>
>          <groupId>org.apache.maven.plugins</groupId>
>          <artifactId>maven-pmd-plugin</artifactId>
>          <version>3.0.1</version>
>          <configuration>
>            <rulesets>
>              <ruleset>/rulesets/java/imports.xml</ruleset>
>            </rulesets>
>          </configuration>
>        </plugin>
> {code}
> mvn pmd:pmd
> tar cvzf imports.tar.gz $(find ./ -name pmd.xml)



--
This message was sent by Atlassian JIRA
(v6.2#6252)