-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49/
-----------------------------------------------------------
Review request for hadoop-common.
Summary
-------
Add listCorruptFileBlocks to FileSystem API. Throw
UnsupportedOperationException for FileSystems that don't support this.
This addresses bug HADOOP-7023.
https://issues.apache.org/jira/browse/HADOOP-7023
Diffs
-----
http://svn.apache.org/repos/asf/hadoop/common/trunk/src/java/org/apache/hadoop/fs/CorruptFileBlocks.java
PRE-CREATION
http://svn.apache.org/repos/asf/hadoop/common/trunk/src/java/org/apache/hadoop/fs/FileSystem.java
1032667
http://svn.apache.org/repos/asf/hadoop/common/trunk/src/java/org/apache/hadoop/fs/FilterFileSystem.java
1032667
http://svn.apache.org/repos/asf/hadoop/common/trunk/src/java/org/apache/hadoop/fs/RawLocalFileSystem.java
1032667
http://svn.apache.org/repos/asf/hadoop/common/trunk/src/java/org/apache/hadoop/fs/ftp/FTPFileSystem.java
1032667
http://svn.apache.org/repos/asf/hadoop/common/trunk/src/java/org/apache/hadoop/fs/kfs/KosmosFileSystem.java
1032667
http://svn.apache.org/repos/asf/hadoop/common/trunk/src/java/org/apache/hadoop/fs/s3/S3FileSystem.java
1032667
http://svn.apache.org/repos/asf/hadoop/common/trunk/src/java/org/apache/hadoop/fs/s3native/NativeS3FileSystem.java
1032667
Diff: https://reviews.apache.org/r/49/diff
Testing
-------
ant test passes
ant test-patch output:
[exec] -1 overall.
[exec]
[exec] +1 @author. The patch does not contain any @author tags.
[exec]
[exec] -1 tests included. The patch doesn't appear to include any new
or modified tests.
[exec] Please justify why no new tests are needed
for this patch.
[exec] Also please list what manual steps were
performed to verify this patch.
[exec]
[exec] +1 javadoc. The javadoc tool did not generate any warning
messages.
[exec]
[exec] +1 javac. The applied patch does not increase the total number
of javac compiler warnings.
[exec]
[exec] -1 findbugs. The patch appears to introduce 2 new Findbugs
warnings.
[exec]
[exec] +1 release audit. The applied patch does not increase the
total number of release audit warnings.
[exec]
[exec] +1 system test framework. The patch passed system test
framework compile.
Note: unit tests for this are included with HDFS-1482.
This patch does not actually introduce any new findbugs warnings (see
MAPREDUCE-2172 for why they show up here).
Thanks,
Patrick
