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

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

Some high level comments:
# Should we split out the patch in common?
# Per discussion under HDFS-8129, in hdfs package let's use 
{{ErasureCodingPolicy}} instead off {{ECPolicy}}
# Can we split out {{ECPolicyLoader}} as follow-on? [~drankye] This class is 
actually not used by other classes except for a test. Did I miss anything?

The rest looks like clean refactor. Will review again after addressing the 
above. Thanks!

> Erasure coding: add ECPolicy to replace schema+cellSize in hadoop-hdfs
> ----------------------------------------------------------------------
>
>                 Key: HDFS-8854
>                 URL: https://issues.apache.org/jira/browse/HDFS-8854
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>    Affects Versions: HDFS-7285
>            Reporter: Walter Su
>            Assignee: Walter Su
>         Attachments: HDFS-8854-HDFS-7285.00.patch, HDFS-8854.00.patch
>
>




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

Reply via email to