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

Andrew Wang commented on HDFS-10918:
------------------------------------

Thanks for working on this Xiao, a few comments:

* I don't think we need that checkAccess call, since the NN will enforce 
permissions already when the client calls getFileInfo
* Is it possible to call dfs.getFileStatus and cast to a HdfsFileStatus, or 
some other way of code sharing? Note that there's extra handling we need to do 
to handle relative paths which is not handled by the current getFeInfo 
implementation. Symlinks also.
* Rather than "getFeInfo" in CryptoAdmin, can we expand to 
"getFileEncryptionInfo" for the user visible flag?
* This probably also needs a doc update

> Add a tool to get FileEncryptionInfo from CLI
> ---------------------------------------------
>
>                 Key: HDFS-10918
>                 URL: https://issues.apache.org/jira/browse/HDFS-10918
>             Project: Hadoop HDFS
>          Issue Type: New Feature
>          Components: encryption
>            Reporter: Xiao Chen
>            Assignee: Xiao Chen
>         Attachments: HDFS-10918.01.patch
>
>




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

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to