[ 
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)

Reply via email to