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 Thu, 14 Jan 2016 21:20:39 GMT

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

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

This is because this jira depends on [HADOOP-12590|https://issues.apache.org/jira/browse/HADOOP-12590]
that is only committed to "trunk":
{noformat}
$ git log --grep HADOOP-12590 asflive/trunk
commit d7ed04758c1bdb1c7caf5cf3a03da3ad81701957
Author: Steve Loughran <stevel@apache.org>
Date:   Sat Jan 9 11:10:20 2016 +0000

    HADOOP-12590. TestCompressorDecompressor failing without stack traces  (John Zhuge via
stevel)
$ git log --grep HADOOP-12590 asflive/branch-2.7
$ git log --grep HADOOP-12590 asflive/branch-2.8
{noformat}

Could someone commit HADOOP-12590 to branch-2.8?

> 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: Minor
>             Fix For: 2.8.0
>
>         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