[ 
https://issues.apache.org/jira/browse/HDFS-8319?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14568959#comment-14568959
 ] 

Walter Su commented on HDFS-8319:
---------------------------------

All good just one minor issue:
We need (5 dataBlks + 1 parityBlk) to recover 1 missed dataBlk. We don't need 
all parityBlks. We need to change this logic:
{code}
618             returnedChunk.state = StripingChunk.MISSING;
...
629             // TODO: close the failed block reader
630             for (int i = 0; i < alignedStripe.chunks.length; i++) {
631               StripingChunk chunk = alignedStripe.chunks[i];
632               if (chunk != null && chunk.state == StripingChunk.REQUESTED) {
633                 readChunk(service, blocks[i], i, corruptedBlockMap);
634               }}}
{code}
How about when missing 1 we add one read? If missing 1 again, we add one again.

> Erasure Coding: support decoding for stateful read
> --------------------------------------------------
>
>                 Key: HDFS-8319
>                 URL: https://issues.apache.org/jira/browse/HDFS-8319
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Jing Zhao
>            Assignee: Jing Zhao
>         Attachments: HDFS-8319.001.patch, HDFS-8319.002.patch
>
>
> HDFS-7678 adds the decoding functionality for pread. This jira plans to add 
> decoding to stateful read.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to