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

Chris Nauroth commented on HDFS-11163:
--------------------------------------

[~surendrasingh], thank you for the patch.  This looks correct to me.  One 
thing I'm unsure about is the potential impact on performance of Mover.  It 
will require an additional {{getStoragePolicy}} RPC per file with the default 
storage policy, whereas previously there was no RPC for those files.  
Unfortunately, I don't see a way to avoid that, at least not with the current 
APIs, because that's how we resolve inheritance of storage policies from parent 
paths.  I would prefer to get an opinion from [~szetszwo].

> Mover should move the file blocks to default storage once policy is unset
> -------------------------------------------------------------------------
>
>                 Key: HDFS-11163
>                 URL: https://issues.apache.org/jira/browse/HDFS-11163
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: balancer & mover
>    Affects Versions: 2.8.0
>            Reporter: Surendra Singh Lilhore
>            Assignee: Surendra Singh Lilhore
>         Attachments: HDFS-11163-001.patch, HDFS-11163-002.patch
>
>
> HDFS-9534 added new API in FileSystem to unset the storage policy. Once 
> policy is unset blocks should move back to the default storage policy.
> Currently mover is not moving file blocks which have zero storage ID
> {code}
>       // currently we ignore files with unspecified storage policy
>       if (policyId == HdfsConstants.BLOCK_STORAGE_POLICY_ID_UNSPECIFIED) {
>         return;
>       }
> {code}



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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to