[
https://issues.apache.org/jira/browse/HADOOP-8444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13284840#comment-13284840
]
Hadoop QA commented on HADOOP-8444:
-----------------------------------
-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12528856/HDFS-3459.patch
against trunk revision .
+1 @author. The patch does not contain any @author tags.
+1 tests included. The patch appears to include 2 new or modified test
files.
-1 patch. The patch command could not apply the patch.
Console output:
https://builds.apache.org/job/PreCommit-HADOOP-Build/1048//console
This message is automatically generated.
> Fix the tests FSMainOperationsBaseTest.java and F
> ileContextMainOperationsBaseTest.java to avoid potential test failure
> -----------------------------------------------------------------------------------------------------------------------
>
> Key: HADOOP-8444
> URL: https://issues.apache.org/jira/browse/HADOOP-8444
> Project: Hadoop Common
> Issue Type: Bug
> Components: fs, test
> Affects Versions: 2.0.0-alpha
> Reporter: Mariappan Asokan
> Attachments: HDFS-3459.patch
>
>
> The above mentioned tests have a path filter to filter file names that
> contain either the string "x" or "X". The filter contains the following if
> statement:
> {code}
> if(file.getName().contains("x") || file.toString().contains("X"))
> {code}
> It should be corrected to:
> {code}
> if(file.getName().contains("x") || file.getName().contains("X"))
> {code}
> Note that toString() returns the full path name. These tests will fail when
> a directory name contains the string "X" and the base file name does not.
> This was the case when we ran the tests on our system.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
