[ https://issues.apache.org/jira/browse/HADOOP-12009?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14640684#comment-14640684 ]
Hudson commented on HADOOP-12009: --------------------------------- SUCCESS: Integrated in Hadoop-Mapreduce-trunk-Java8 #263 (See [https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/263/]) HADOOP-12009: Clarify FileSystem.listStatus() sorting order & fix FileSystemContractBaseTest:testListStatus. (J.Andreina via jghoman) (jghoman: rev ab3197c20452e0dd908193d6854c204e6ee34645) * hadoop-common-project/hadoop-common/src/site/markdown/filesystem/filesystem.md * hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FileSystem.java * hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/fs/FileSystemContractBaseTest.java * hadoop-common-project/hadoop-common/CHANGES.txt > Clarify FileSystem.listStatus() sorting order & fix > FileSystemContractBaseTest:testListStatus > ---------------------------------------------------------------------------------------------- > > Key: HADOOP-12009 > URL: https://issues.apache.org/jira/browse/HADOOP-12009 > Project: Hadoop Common > Issue Type: Improvement > Components: test > Reporter: Jakob Homan > Assignee: J.Andreina > Priority: Minor > Fix For: 3.0.0 > > Attachments: HADOOP-12009-003.patch, HADOOP-12009-004.patch, > HADOOP-12009.1.patch > > > FileSystem.listStatus does not guarantee that implementations will return > sorted entries: > {code} /** > * List the statuses of the files/directories in the given path if the path > is > * a directory. > * > * @param f given path > * @return the statuses of the files/directories in the given patch > * @throws FileNotFoundException when the path does not exist; > * IOException see specific implementation > */ > public abstract FileStatus[] listStatus(Path f) throws > FileNotFoundException, > IOException;{code} > However, FileSystemContractBaseTest, expects the elements to come back sorted: > {code} Path[] testDirs = { path("/test/hadoop/a"), > path("/test/hadoop/b"), > path("/test/hadoop/c/1"), }; > > // ... > paths = fs.listStatus(path("/test/hadoop")); > assertEquals(3, paths.length); > assertEquals(path("/test/hadoop/a"), paths[0].getPath()); > assertEquals(path("/test/hadoop/b"), paths[1].getPath()); > assertEquals(path("/test/hadoop/c"), paths[2].getPath());{code} > We should pass this test as long as all the paths are there, regardless of > their ordering. -- This message was sent by Atlassian JIRA (v6.3.4#6332)