[ https://issues.apache.org/jira/browse/HDFS-8853?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14697873#comment-14697873 ]
Zhe Zhang commented on HDFS-8853: --------------------------------- Thanks [~andreina] for the patch. Do you mind rebasing it? I was also thinking about the issue when creating HDFS-8833 patch. In the long term, it might be better for the client to pass a {{String}} to NN instead of the actual policy/schema. > Erasure Coding: Provide ECSchema validation when creating ECZone > ---------------------------------------------------------------- > > Key: HDFS-8853 > URL: https://issues.apache.org/jira/browse/HDFS-8853 > Project: Hadoop HDFS > Issue Type: Sub-task > Reporter: Rakesh R > Assignee: J.Andreina > Attachments: HDFS-8853-HDFS-7285-01.patch > > > Presently the {{DFS#createErasureCodingZone(path, ecSchema, cellSize)}} > doesn't have any validation that the given {{ecSchema}} is available in > {{ErasureCodingSchemaManager#activeSchemas}} list. Now, if it doesn't exists > then will create the ECZone with {{null}} schema. IMHO we could improve this > by doing necessary basic sanity checks. -- This message was sent by Atlassian JIRA (v6.3.4#6332)