[
https://issues.apache.org/jira/browse/HADOOP-12700?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
John Zhuge updated HADOOP-12700:
--------------------------------
Description:
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.
was:
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 code are
not checked by checkstyle by default. Maven checkstyle plugin parameter
*includeTestSourceDirectory* is *false* by default.
> 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)
