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

Arpit Agarwal commented on HDFS-8345:
-------------------------------------

bq. That's not so bad. Is there a reason not to include a convenience method 
that just takes a BlockStoragePolicySpi object directly?
That doesn't sound useful enough to introduce one more FileSystem API. Can you 
think of an example where the extra API call would simplify application code?

bq. Can you point me to the exposure? BlockStoragePolicy is IA.Private, and I 
would expect we can use covariant return types to make a source-compatible 
update.
It was tagged IA.Private but returned via a LimitedPrivate interface, 
unfortunate as that is. Changing the name, package or public interface of 
BlockStoragePolicy would technically be incompatible. However if you suggest a 
better name for the new interface I'd be happy to change it.

bq. The one in the docs: 
http://hadoop.apache.org/docs/current/hadoop-project-dist/hadoop-common/filesystem/index.html
Okay let me file a separate Jira for that.

> Storage policy APIs must be exposed via the FileSystem interface
> ----------------------------------------------------------------
>
>                 Key: HDFS-8345
>                 URL: https://issues.apache.org/jira/browse/HDFS-8345
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: hdfs-client
>    Affects Versions: 2.7.0
>            Reporter: Arpit Agarwal
>            Assignee: Arpit Agarwal
>              Labels: BB2015-05-TBR
>         Attachments: HDFS-8345.01.patch, HDFS-8345.02.patch, 
> HDFS-8345.03.patch, HDFS-8345.04.patch, HDFS-8345.05.patch, HDFS-8345.06.patch
>
>
> The storage policy APIs are not exposed via FileSystem. Since 
> DistributedFileSystem is tagged as LimitedPrivate we should expose the APIs 
> through FileSystem for use by other applications.



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

Reply via email to