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

Zhe Zhang commented on HDFS-8186:
---------------------------------

Thanks Walter for updating the patch! +1 on the latest version pending the 
following minor issues. After fixing the issues please feel free to commit the 
patch.
# Looks like we should add a TODO that webhdfs on striped files will be 
supported later?
{code}
   public DatanodeStorageInfo[] chooseTarget4WebHDFS(String src,
       DatanodeDescriptor clientnode, Set<Node> excludes, long blocksize) {
-    return blockplacement.chooseTarget(src, 1, clientnode,
+    return placementPolicies.getPolicy(false).chooseTarget(src, 1, client node,
{code}
# {{FSNamesystem}} needs a rebase

> Erasure coding: Make block placement policy for EC file configurable
> --------------------------------------------------------------------
>
>                 Key: HDFS-8186
>                 URL: https://issues.apache.org/jira/browse/HDFS-8186
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Walter Su
>            Assignee: Walter Su
>         Attachments: HDFS-8186-HDFS-7285.002.patch, HDFS-8186.001.patch
>
>
> This includes:
> 1. User can config block placement policy for EC file in xml configuration 
> file.
> 2. EC policy works for EC file, replication policy works for non-EC file. 
> They are coexistent.
> Not includes:
> 1. Details of block placement policy for EC. Discussion and implementation 
> goes to HDFS-7613.



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

Reply via email to