[
https://issues.apache.org/jira/browse/HDFS-9103?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
James Clampffer updated HDFS-9103:
----------------------------------
Attachment: HDFS-9103.HDFS-8707.010.patch
Thanks for the clarification [~wheat9]
New patch posted:
-got bad_datanode_tracker.h out of the public headers, moved into lib/fs
because thats where it's most tightly coupled
-declared NodeExclusionRule in hdfs.h
-got rid of comment about static cast
-renamed 'optional_exclude_rule' param to 'excluded_nodes'
-inlined SelectBlockAndNode
-changed dn selection to use a find_if rather than an explicit loop
-kept existing bad_datanode_test tests
-put the unit tests for BadDataNodeTracker and ExclusionSet that don't use mock
objects/methods into a seperate test and cmake target
Other things:
-NodeExclusionRule and classes that derive from it got virtual destructors to
avoid leaks
-Added tests for the ExcludedSet object. It's incredibly simple but more tests
won't hurt.
> Retry reads on DN failure
> -------------------------
>
> Key: HDFS-9103
> URL: https://issues.apache.org/jira/browse/HDFS-9103
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: hdfs-client
> Reporter: Bob Hansen
> Assignee: James Clampffer
> Fix For: HDFS-8707
>
> Attachments: HDFS-9103.1.patch, HDFS-9103.2.patch,
> HDFS-9103.HDFS-8707.006.patch, HDFS-9103.HDFS-8707.007.patch,
> HDFS-9103.HDFS-8707.008.patch, HDFS-9103.HDFS-8707.009.patch,
> HDFS-9103.HDFS-8707.010.patch, HDFS-9103.HDFS-8707.3.patch,
> HDFS-9103.HDFS-8707.4.patch, HDFS-9103.HDFS-8707.5.patch
>
>
> When AsyncPreadSome fails, add the failed DataNode to the excluded list and
> try again.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
