[ https://issues.apache.org/jira/browse/HDFS-7866?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15184300#comment-15184300 ]
Rui Li commented on HDFS-7866: ------------------------------ Thanks Walter for the suggestions. bq. separating the logic of manipulation of the 12-bits. 2 sets of set/get methods for them. It makes sense to have 2 separate sets of get/set methods. But if we do it here, we'll be just repeating the code. What do you think? bq. We are sure policyID is strictly <=7-bits right? I think 128 policies is enough for now, and we can just consider negative ID as invalid. If we have more policies in the future, we can support negative IDs which means we can have 256 different IDs. Then we can think about what to return if we call getErasureCodingPolicyID for non-EC files. Maybe throw an exception? [~zhz], please hold on the patch. The latest {{TestEditLog}} failure may be related. I'll look into it. > Erasure coding: NameNode manages multiple erasure coding policies > ----------------------------------------------------------------- > > Key: HDFS-7866 > URL: https://issues.apache.org/jira/browse/HDFS-7866 > Project: Hadoop HDFS > Issue Type: Sub-task > Reporter: Kai Zheng > Assignee: Rui Li > Attachments: HDFS-7866-v1.patch, HDFS-7866-v2.patch, > HDFS-7866-v3.patch, HDFS-7866.10.patch, HDFS-7866.11.patch, > HDFS-7866.12.patch, HDFS-7866.4.patch, HDFS-7866.5.patch, HDFS-7866.6.patch, > HDFS-7866.7.patch, HDFS-7866.8.patch, HDFS-7866.9.patch > > > This is to extend NameNode to load, list and sync predefine EC schemas in > authorized and controlled approach. The provided facilities will be used to > implement DFSAdmin commands so admin can list available EC schemas, then > could choose some of them for target EC zones. -- This message was sent by Atlassian JIRA (v6.3.4#6332)