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

Rakesh R commented on HDFS-8266:
--------------------------------

Hi All,

I've one observation while writing snapshot related unit test case. 
{{ErasureCodingZoneManager#getECZoneInfo()}} function is not properly resolving 
the {{inode.getFullPathName()}}, its returning the path including the snapshot 
name {{/zone/.snapshot/snap1}}. I feel we could improve this by returning only 
path {{/zone}}. Shall I go ahead by changing this logic like,

{code}
 return new 
ErasureCodingZoneInfo(dir.getInode(inode.getId()).getFullPathName(), schema);
{code}

> Erasure Coding: Test of snapshot/.trash with EC files
> -----------------------------------------------------
>
>                 Key: HDFS-8266
>                 URL: https://issues.apache.org/jira/browse/HDFS-8266
>             Project: Hadoop HDFS
>          Issue Type: Test
>    Affects Versions: HDFS-7285
>            Reporter: GAO Rui
>         Attachments: HDFS-8266-HDFS-7285-00.patch
>
>




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

Reply via email to