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

Jing Zhao commented on HDFS-8489:
---------------------------------

+1 for the 05 patch. Since the only change between 04 and 05 is removing the 
"protected" of the two methods, I will not wait for the Jenkins run.

> Subclass BlockInfo to represent contiguous blocks
> -------------------------------------------------
>
>                 Key: HDFS-8489
>                 URL: https://issues.apache.org/jira/browse/HDFS-8489
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: namenode
>    Affects Versions: 2.7.0
>            Reporter: Zhe Zhang
>            Assignee: Zhe Zhang
>         Attachments: HDFS-8489.00.patch, HDFS-8489.01.patch, 
> HDFS-8489.02.patch, HDFS-8489.03.patch, HDFS-8489.04.patch, HDFS-8489.05.patch
>
>
> As second step of the cleanup, we should make {{BlockInfo}} an abstract class 
> and merge the subclass {{BlockInfoContiguous}} from HDFS-7285 into trunk. The 
> patch should clearly separate where to use the abstract class versus the 
> subclass.



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

Reply via email to