[ 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)