You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-dev@hadoop.apache.org by Patrick Kling <pk...@cs.uwaterloo.ca> on 2010/11/03 04:23:53 UTC

Review Request: Add boolean field isCorrupt to BlockLocation

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26/
-----------------------------------------------------------

Review request for hadoop-common.


Summary
-------

Add boolean field isCorrupt to BlockLocation

This is needed to allow DFSClient.getBlockLocations to notify the calling application when returning a BlockLocation that corresponds to a corrupt block. Currently, this happens when there are no uncorrupted replicas of a requested block.


This addresses bug HADOOP-7013.
    https://issues.apache.org/jira/browse/HADOOP-7013


Diffs
-----

  http://svn.apache.org/repos/asf/hadoop/common/trunk/src/java/org/apache/hadoop/fs/BlockLocation.java 1030319 

Diff: https://reviews.apache.org/r/26/diff


Testing
-------


Thanks,

Patrick

Re: Review Request: Add boolean field isCorrupt to BlockLocation

Posted by Patrick Kling <pk...@cs.uwaterloo.ca>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26/
-----------------------------------------------------------

(Updated 2010-11-02 21:13:48.059172)


Review request for hadoop-common.


Summary
-------

Add boolean field isCorrupt to BlockLocation

This is needed to allow DFSClient.getBlockLocations to notify the calling application when returning a BlockLocation that corresponds to a corrupt block. Currently, this happens when there are no uncorrupted replicas of a requested block.


This addresses bug HADOOP-7013.
    https://issues.apache.org/jira/browse/HADOOP-7013


Diffs
-----

  http://svn.apache.org/repos/asf/hadoop/common/trunk/src/java/org/apache/hadoop/fs/BlockLocation.java 1030319 

Diff: https://reviews.apache.org/r/26/diff


Testing (updated)
-------

TestDFSUtil in HDFS-1483


Thanks,

Patrick