[
https://issues.apache.org/jira/browse/HADOOP-12161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14638218#comment-14638218
]
Vinayakumar B commented on HADOOP-12161:
----------------------------------------
bq. 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
This seems to be good. whether needs to update the cached policies periodically?
How about adding a separate protocol method {{getStoragePolicy()}} in
ClientProtocol similar to {{setStoragePolicy()}} dedicated for this purpose? I
feel it just looks good to get in terms of code, since this is going to be a
public API, and no. of RPCs will be 1.
I feel this improvement could go in separate followup jira in HDFS, agree?
If agree, I am also +1 for 004 patch.
> 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)
