[
https://issues.apache.org/jira/browse/HDFS-7743?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14310176#comment-14310176
]
Daryn Sharp commented on HDFS-7743:
-----------------------------------
What is the rationale for renaming to {{BlockInfoContiguous}}? It might make
sense if {{BlockInfo}} became a common super-class for various impls like
contiguous, sparse, etc. Since the patch also renames all the users of the
object, it seems like this was a rename for the sake of renaming? If yes, I'd
please request refraining since I'm extremely close to posting a reference-free
block triplets replacement and would like to avoid unnecessary merge issues.
> Code cleanup of BlockInfo and rename BlockInfo to BlockReplicationInfo
> ----------------------------------------------------------------------
>
> Key: HDFS-7743
> URL: https://issues.apache.org/jira/browse/HDFS-7743
> Project: Hadoop HDFS
> Issue Type: Improvement
> Components: namenode
> Reporter: Jing Zhao
> Assignee: Jing Zhao
> Priority: Minor
> Attachments: HDFS-7743.000.patch, HDFS-7743.001.patch,
> HDFS-7743.002.patch
>
>
> In the work of erasure coding (HDFS-7285), we plan to extend the class
> BlockInfo to two subclasses: BlockReplicationInfo and BlockGroupInfo
> (HDFS-7716). To ease the HDFS-EC branch syncing with trunk, this jira plans
> to rename the current BlockInfo to BlockReplicationInfo in trunk. In the
> meanwhile, we can also use this chance to do some minor code cleanup.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
