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

Arpit Agarwal edited comment on HADOOP-12161 at 7/22/15 6:27 PM:
-----------------------------------------------------------------

+1 for the v004 patch. I am still concerned about requiring two RPCs per call 
but I don't see an easy workaround. A mitigation could be for the DFSClient to 
cache the policies locally. I will delay committing until tomorrow in case 
[~vinayrpet]/[~jingzhao] have more comments.

Thanks for your patience with this [~brahmareddy].


was (Author: arpitagarwal):
+1 for the patch. I am still concerned about requiring two RPCs per call but I 
don't see an easy workaround for it. A mitigation could be for the DFSClient to 
cache the policies locally. I will delay committing until tomorrow in case 
[~vinayrpet]/[~jingzhao] have more comments.

Thanks for your patience with this [~brahmareddy].

> Add getStoragePolicy API to the FileSystem interface
> ----------------------------------------------------
>
>                 Key: HADOOP-12161
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12161
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Arpit Agarwal
>            Assignee: Brahma Reddy Battula
>         Attachments: HADOOP-12161-001.patch, HADOOP-12161-002.patch, 
> HADOOP-12161-003.patch, HADOOP-12161-004.patch
>
>
> HDFS-8345 added {{FileSystem#getAllStoragePolicies}} and 
> {{FileSystem#setStoragePolicy}}. Jira to
> # Add a corresponding {{FileSystem#getStoragePolicy}} to query the storage 
> policy for a given file/directory.
> # Add corresponding implementation for HDFS i.e. 
> {{DistributedFileSystem#getStoragePolicy}}.
> # Update the [FileSystem 
> specification|https://hadoop.apache.org/docs/current/hadoop-project-dist/hadoop-common/filesystem/filesystem.html].
>  This will require editing 
> _hadoop-common-project/hadoop-common/src/site/markdown/filesystem/filesystem.md_.



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

Reply via email to