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

Zhe Zhang commented on HDFS-7716:
---------------------------------

Thanks Jing! The high-level plan looks good. Below are some detailed items we 
can further discuss:
# Shall we use {{triplets}} to record block locations in an block group?
# If we use {{triplets}} for both replicated blocks and EC block groups, I 
don't clearly see which fields and methods to offload to 
{{BlockReplicationInfo}}. Thoughts?
# We discussed recording the EC schema in {{BlockGroupInfo}} to avoid 
frequently checking INode. Before HDFS-7337 we can simply record the number of 
data and parity blocks (2 {{short}} variables). 
# An INode feature for {{BlockGroupInfo}} sounds good. Before our meetup last 
week, HDFS-7339's [patch | 
https://issues.apache.org/jira/secure/attachment/12693731/HDFS-7339-006.patch] 
used that approach. We might be able to reuse some code.
# [~andrew.wang] and I have discussed how to support snapshots when a file is 
converted between replicated and EC forms. If we keep both the regular 
{{BlockInfo}} array and the {{BlockGroupInfo}} feature, we should be able to 
calculate snapshots.

> Erasure Coding: extend BlockInfo to handle EC info
> --------------------------------------------------
>
>                 Key: HDFS-7716
>                 URL: https://issues.apache.org/jira/browse/HDFS-7716
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Jing Zhao
>            Assignee: Jing Zhao
>
> The current BlockInfo's implementation only supports the replication 
> mechanism. To use the same blocksMap handling block group and its data/parity 
> blocks, we need to define a new BlockGroupInfo class.



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

Reply via email to