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

Walter Su commented on HDFS-8441:
---------------------------------

>... Maybe we could add a method in {{FSDirectory#isInECZone(src)}} and use it 
>to tell the condition?
Good idea. I found {{dir#isInECZone(iip)}} exists. So I add 
{{ns#isInEcZone(src)}}. I think usually {{ns}} takes {{src}} arg, and {{dir}} 
takes {{iip}}.
>... Please add message to the fail() statement like...
Done.
Thank you both. Uploaded 003 patch.

> Erasure Coding: make condition check earlier for setReplication
> ---------------------------------------------------------------
>
>                 Key: HDFS-8441
>                 URL: https://issues.apache.org/jira/browse/HDFS-8441
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Walter Su
>            Assignee: Walter Su
>         Attachments: HDFS-8441-HDFS-7285.001.patch, 
> HDFS-8441-HDFS-7285.002.patch, HDFS-8441-HDFS-7285.003.patch
>
>
> Changes:
> 1. {{UnsupportedActionException}} is more user-firendly.
> 2. check condition before update storage count
> 3. support create EC file with replication=0



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

Reply via email to