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 Mon, 11 Jan 2016 01:46:39 GMT

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

John Zhuge commented on HADOOP-12700:

Filed [HADOOP-12701|https://issues.apache.org/jira/browse/HADOOP-12701] to run checkstyle
on test source files.

> 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
> 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.

This message was sent by Atlassian JIRA

View raw message