hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "John Zhuge (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-12700) Remove unused import in TestCompressorDecompressor.java
Date Tue, 12 Jan 2016 15:27:40 GMT

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

John Zhuge commented on HADOOP-12700:
-------------------------------------

[~stevel@apache.org],

Could you please review this quick fix, a followup to [HADOOP-12590]?

Thanks,
John.

> Remove unused import in TestCompressorDecompressor.java
> -------------------------------------------------------
>
>                 Key: HADOOP-12700
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12700
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: John Zhuge
>            Assignee: John Zhuge
>            Priority: Trivial
>         Attachments: HADOOP-12700.001.patch
>
>
> The fix for [HADOOP-12590|https://issues.apache.org/jira/browse/HADOOP-12590] left an
unused import in TestCompressorDecompressor.java.
> After uploading the patch for HADOOP-12590, I spotted the problem in IntelliJ which marked
the unused import *gray*, but it was too late.
> The problem was not detected by precommit check because test source files are not checked
by checkstyle by default. Maven checkstyle plugin parameter *includeTestSourceDirectory* is
*false* by default.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message