[ https://issues.apache.org/jira/browse/HDFS-1765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13168042#comment-13168042 ]
Uma Maheswara Rao G commented on HDFS-1765: ------------------------------------------- Thanks a lot Eli, for the reviews! Updated the patch. Functionally same as last patch with addressing the comments from Eli. TestReplicationPolicy: * rename chooseUnderReplicatedBlocks to chosenBlocks Done * The new testChooseUnderReplicatedBlocks looks great Thanks Eli. :-) UnderReplicatedBlocks: * Can remove the " // replicate blocks" comment since we're not actually replicating blocks here. Actually this is in BlockManager and was a old comment. Just removed it. * The comment in chooseUnderReplicatedBlocks references priorityVsReplIndex instead of priorityToReplIdx couldn't figure out in last patch. Done. Thanks for catching it. * "is no blocks" -> "are no blocks" " "full fill" -> "fulfill" - done. Thanks Uma > Block Replication should respect under-replication block priority > ----------------------------------------------------------------- > > Key: HDFS-1765 > URL: https://issues.apache.org/jira/browse/HDFS-1765 > Project: Hadoop HDFS > Issue Type: Improvement > Components: name-node > Affects Versions: 0.23.0 > Reporter: Hairong Kuang > Assignee: Uma Maheswara Rao G > Fix For: 0.24.0 > > Attachments: HDFS-1765.patch, HDFS-1765.patch, HDFS-1765.patch, > HDFS-1765.patch, HDFS-1765.pdf, underReplicatedQueue.pdf > > Time Spent: 0.5h > Remaining Estimate: 0h > > Currently under-replicated blocks are assigned different priorities depending > on how many replicas a block has. However the replication monitor works on > blocks in a round-robin fashion. So the newly added high priority blocks > won't get replicated until all low-priority blocks are done. One example is > that on decommissioning datanode WebUI we often observe that "blocks with > only decommissioning replicas" do not get scheduled to replicate before other > blocks, so risking data availability if the node is shutdown for repair > before decommission completes. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira