hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vinayakumar B (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-12161) Add getStoragePolicy API to the FileSystem interface
Date Thu, 23 Jul 2015 06:06:05 GMT

    [ 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,
> 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

View raw message