Improve listFiles API introduced by HADOOP-6870
-----------------------------------------------
Key: HADOOP-6890
URL: https://issues.apache.org/jira/browse/HADOOP-6890
Project: Hadoop Common
Issue Type: Improvement
Components: fs
Affects Versions: 0.22.0
Reporter: Hairong Kuang
Assignee: Hairong Kuang
Fix For: 0.22.0
This jira is mainly for addressing Suresh's review comments for HADOOP-6870:
1. General comment: I have concerns about recursive listing. This could be
abused by the applications, creating a lot of requests into HDFS.
2. Any deletion of files/directories while reursing through directories
results in RuntimeException and application has a partial result. Should we
ignore if a directory was in stack and was not found later when iterating
through it?
3. FileSystem.java
* listFile() - method javadoc could be better organized - first write
about if path is directory and two cases recursive=true and false. Then if path
is file and two cases recursive=true or false.
* listFile() - document throwing RuntimeException,
UnsupportedOperationException and the possible cause. IOException is no longer
thrown.
4. TestListFiles.java
* testDirectory() - comments test empty directory and test directory
with 1 file should be moved up to relevant sections of the test.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.