[jira] [Commented] (HADOOP-9624) TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name
[ https://issues.apache.org/jira/browse/HADOOP-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13687852#comment-13687852 ] Hudson commented on HADOOP-9624: Integrated in Hadoop-Yarn-trunk #245 (See [https://builds.apache.org/job/Hadoop-Yarn-trunk/245/]) HADOOP-9624. TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name. Contributed by Xi Fang. (Revision 1494363) Result = FAILURE cnauroth : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVNview=revrev=1494363 Files : * /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt * /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/FSMainOperationsBaseTest.java * /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/FileContextMainOperationsBaseTest.java TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name - Key: HADOOP-9624 URL: https://issues.apache.org/jira/browse/HADOOP-9624 Project: Hadoop Common Issue Type: Test Components: test Affects Versions: 3.0.0, 1-win, 2.1.0-beta, 1.3.0 Environment: Windows Reporter: Xi Fang Assignee: Xi Fang Priority: Minor Labels: test Fix For: 3.0.0, 1-win, 2.1.0-beta, 1.3.0 Attachments: HADOOP-9624.branch-1.2.patch, HADOOP-9624.branch-1.patch, HADOOP-9624.patch, HADOOP-9624.trunk.patch TestFSMainOperationsLocalFileSystem extends Class FSMainOperationsBaseTest. PathFilter FSMainOperationsBaseTest#TEST_X_FILTER checks if a path has x and X in its name. {code} final private static PathFilter TEST_X_FILTER = new PathFilter() { public boolean accept(Path file) { if(file.getName().contains(x) || file.toString().contains(X)) return true; else return false; {code} Some of the test cases construct a path by combining path TEST_ROOT_DIR with a customized partial path. The problem is that TEST_ROOT_DIR may also has X in its name. The path check will pass even if the customized partial path doesn't have X. However, for this case the path filter is supposed to reject this path. An easy fix is to change file.toString().contains(X) to file.getName().contains(X). Note that org.apache.hadoop.fs.Path.getName() only returns the final component of this path. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9624) TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name
[ https://issues.apache.org/jira/browse/HADOOP-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13687998#comment-13687998 ] Hudson commented on HADOOP-9624: Integrated in Hadoop-Mapreduce-trunk #1462 (See [https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1462/]) HADOOP-9624. TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name. Contributed by Xi Fang. (Revision 1494363) Result = FAILURE cnauroth : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVNview=revrev=1494363 Files : * /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt * /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/FSMainOperationsBaseTest.java * /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/FileContextMainOperationsBaseTest.java TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name - Key: HADOOP-9624 URL: https://issues.apache.org/jira/browse/HADOOP-9624 Project: Hadoop Common Issue Type: Test Components: test Affects Versions: 3.0.0, 1-win, 2.1.0-beta, 1.3.0 Environment: Windows Reporter: Xi Fang Assignee: Xi Fang Priority: Minor Labels: test Fix For: 3.0.0, 1-win, 2.1.0-beta, 1.3.0 Attachments: HADOOP-9624.branch-1.2.patch, HADOOP-9624.branch-1.patch, HADOOP-9624.patch, HADOOP-9624.trunk.patch TestFSMainOperationsLocalFileSystem extends Class FSMainOperationsBaseTest. PathFilter FSMainOperationsBaseTest#TEST_X_FILTER checks if a path has x and X in its name. {code} final private static PathFilter TEST_X_FILTER = new PathFilter() { public boolean accept(Path file) { if(file.getName().contains(x) || file.toString().contains(X)) return true; else return false; {code} Some of the test cases construct a path by combining path TEST_ROOT_DIR with a customized partial path. The problem is that TEST_ROOT_DIR may also has X in its name. The path check will pass even if the customized partial path doesn't have X. However, for this case the path filter is supposed to reject this path. An easy fix is to change file.toString().contains(X) to file.getName().contains(X). Note that org.apache.hadoop.fs.Path.getName() only returns the final component of this path. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9624) TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name
[ https://issues.apache.org/jira/browse/HADOOP-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13687347#comment-13687347 ] Hadoop QA commented on HADOOP-9624: --- {color:red}-1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12588475/HADOOP-9624.branch-1.2.patch against trunk revision . {color:red}-1 patch{color}. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2669//console This message is automatically generated. TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name - Key: HADOOP-9624 URL: https://issues.apache.org/jira/browse/HADOOP-9624 Project: Hadoop Common Issue Type: Test Components: test Affects Versions: 3.0.0, 1-win, 2.1.0-beta, 1.3.0 Environment: Windows Reporter: Xi Fang Assignee: Xi Fang Priority: Minor Labels: test Attachments: HADOOP-9624.branch-1.2.patch, HADOOP-9624.branch-1.patch, HADOOP-9624.patch, HADOOP-9624.trunk.patch TestFSMainOperationsLocalFileSystem extends Class FSMainOperationsBaseTest. PathFilter FSMainOperationsBaseTest#TEST_X_FILTER checks if a path has x and X in its name. {code} final private static PathFilter TEST_X_FILTER = new PathFilter() { public boolean accept(Path file) { if(file.getName().contains(x) || file.toString().contains(X)) return true; else return false; {code} Some of the test cases construct a path by combining path TEST_ROOT_DIR with a customized partial path. The problem is that TEST_ROOT_DIR may also has X in its name. The path check will pass even if the customized partial path doesn't have X. However, for this case the path filter is supposed to reject this path. An easy fix is to change file.toString().contains(X) to file.getName().contains(X). Note that org.apache.hadoop.fs.Path.getName() only returns the final component of this path. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9624) TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name
[ https://issues.apache.org/jira/browse/HADOOP-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13687359#comment-13687359 ] Hudson commented on HADOOP-9624: Integrated in Hadoop-trunk-Commit #3972 (See [https://builds.apache.org/job/Hadoop-trunk-Commit/3972/]) HADOOP-9624. TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name. Contributed by Xi Fang. (Revision 1494363) Result = SUCCESS cnauroth : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVNview=revrev=1494363 Files : * /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt * /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/FSMainOperationsBaseTest.java * /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/FileContextMainOperationsBaseTest.java TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name - Key: HADOOP-9624 URL: https://issues.apache.org/jira/browse/HADOOP-9624 Project: Hadoop Common Issue Type: Test Components: test Affects Versions: 3.0.0, 1-win, 2.1.0-beta, 1.3.0 Environment: Windows Reporter: Xi Fang Assignee: Xi Fang Priority: Minor Labels: test Attachments: HADOOP-9624.branch-1.2.patch, HADOOP-9624.branch-1.patch, HADOOP-9624.patch, HADOOP-9624.trunk.patch TestFSMainOperationsLocalFileSystem extends Class FSMainOperationsBaseTest. PathFilter FSMainOperationsBaseTest#TEST_X_FILTER checks if a path has x and X in its name. {code} final private static PathFilter TEST_X_FILTER = new PathFilter() { public boolean accept(Path file) { if(file.getName().contains(x) || file.toString().contains(X)) return true; else return false; {code} Some of the test cases construct a path by combining path TEST_ROOT_DIR with a customized partial path. The problem is that TEST_ROOT_DIR may also has X in its name. The path check will pass even if the customized partial path doesn't have X. However, for this case the path filter is supposed to reject this path. An easy fix is to change file.toString().contains(X) to file.getName().contains(X). Note that org.apache.hadoop.fs.Path.getName() only returns the final component of this path. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9624) TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name
[ https://issues.apache.org/jira/browse/HADOOP-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13687378#comment-13687378 ] Chris Nauroth commented on HADOOP-9624: --- I committed the trunk patch to trunk, branch-2, and branch-2.1-beta. I committed the branch-1 patch to branch-1 and branch-1-win. Xi, thank you for contributing this test fix! TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name - Key: HADOOP-9624 URL: https://issues.apache.org/jira/browse/HADOOP-9624 Project: Hadoop Common Issue Type: Test Components: test Affects Versions: 3.0.0, 1-win, 2.1.0-beta, 1.3.0 Environment: Windows Reporter: Xi Fang Assignee: Xi Fang Priority: Minor Labels: test Attachments: HADOOP-9624.branch-1.2.patch, HADOOP-9624.branch-1.patch, HADOOP-9624.patch, HADOOP-9624.trunk.patch TestFSMainOperationsLocalFileSystem extends Class FSMainOperationsBaseTest. PathFilter FSMainOperationsBaseTest#TEST_X_FILTER checks if a path has x and X in its name. {code} final private static PathFilter TEST_X_FILTER = new PathFilter() { public boolean accept(Path file) { if(file.getName().contains(x) || file.toString().contains(X)) return true; else return false; {code} Some of the test cases construct a path by combining path TEST_ROOT_DIR with a customized partial path. The problem is that TEST_ROOT_DIR may also has X in its name. The path check will pass even if the customized partial path doesn't have X. However, for this case the path filter is supposed to reject this path. An easy fix is to change file.toString().contains(X) to file.getName().contains(X). Note that org.apache.hadoop.fs.Path.getName() only returns the final component of this path. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (HADOOP-9624) TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name
[ https://issues.apache.org/jira/browse/HADOOP-9624?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13687389#comment-13687389 ] Xi Fang commented on HADOOP-9624: - Chris, thanks for your review and comments. TestFSMainOperationsLocalFileSystem failed when the Hadoop test root path has X in its name - Key: HADOOP-9624 URL: https://issues.apache.org/jira/browse/HADOOP-9624 Project: Hadoop Common Issue Type: Test Components: test Affects Versions: 3.0.0, 1-win, 2.1.0-beta, 1.3.0 Environment: Windows Reporter: Xi Fang Assignee: Xi Fang Priority: Minor Labels: test Fix For: 3.0.0, 1-win, 2.1.0-beta, 1.3.0 Attachments: HADOOP-9624.branch-1.2.patch, HADOOP-9624.branch-1.patch, HADOOP-9624.patch, HADOOP-9624.trunk.patch TestFSMainOperationsLocalFileSystem extends Class FSMainOperationsBaseTest. PathFilter FSMainOperationsBaseTest#TEST_X_FILTER checks if a path has x and X in its name. {code} final private static PathFilter TEST_X_FILTER = new PathFilter() { public boolean accept(Path file) { if(file.getName().contains(x) || file.toString().contains(X)) return true; else return false; {code} Some of the test cases construct a path by combining path TEST_ROOT_DIR with a customized partial path. The problem is that TEST_ROOT_DIR may also has X in its name. The path check will pass even if the customized partial path doesn't have X. However, for this case the path filter is supposed to reject this path. An easy fix is to change file.toString().contains(X) to file.getName().contains(X). Note that org.apache.hadoop.fs.Path.getName() only returns the final component of this path. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira