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

Zhe Zhang commented on HDFS-8854:
---------------------------------

Thanks Walter for taking this task. I think we need to separate the discussion 
into 2 questions:
# Whether we need an abstraction for current ECSchema + cellSize. I think we 
clearly do need it. You have a good summary of the advantages:
{quote}
Advantages:
1. shorter args list, cleaner code.
2. store one field in BlockInfoStriped, instead of two fields. Less memory 
usage.
{quote}
# Naming of the new abstraction. We probably shouldn't call it a schema, 
because _schema_ is already a basic concept in codec. I think 
{{ErasureCodingPolicy}} is a better name. 

Maybe we can keep {{ECSchema}} class in COMMON, and refactor HDFS codes to use 
{{ErasureCodingPolicy}} instead?

> Erasure coding: Move cellSize inside ECSchema
> ---------------------------------------------
>
>                 Key: HDFS-8854
>                 URL: https://issues.apache.org/jira/browse/HDFS-8854
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Walter Su
>            Assignee: Walter Su
>




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

Reply via email to