[
https://issues.apache.org/jira/browse/HADOOP-12161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14625267#comment-14625267
]
Arpit Agarwal commented on HADOOP-12161:
----------------------------------------
Thanks for updating the patch [~brahmareddy]. The changes look good.
Perhaps we can make the Javadoc little more descriptive since it may not be
obvious what the {{byte}} return type means. e.g. _Query the effective storage
policy ID for the given file or directory. The storage policy ID to storage
policy name mapping for the filesystem implementation can be retrieved using
\{@link #getAllStoragePolicies\}_
+1 from me otherwise. [~jingzhao], do you have any comments on this API since
you worked on the storage policy implementation? Thanks.
> 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
>
>
> 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)
