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

Akira AJISAKA commented on HDFS-8462:
-------------------------------------

Thanks [~jagadesh.kiran] for taking this issue. Some comments:
{code:title=FSImageLoader#getXAttrs}
   * @return JSON formatted ListXAttrs
{code}
1. Would you change {{ListXAttrs}} to {{XAttrs}}? 
(http://hadoop.apache.org/docs/r2.7.0/hadoop-project-dist/hadoop-hdfs/WebHDFS.html#XAttrs_JSON_Schema)

{code:title=FSImageLoader#listXAttrs}
   * @return JSON formatted ListXAttrs
{code}
2. Would you change {{ListXAttrs}} to {{XAttrNames}}? 
(http://hadoop.apache.org/docs/r2.7.0/hadoop-project-dist/hadoop-hdfs/WebHDFS.html#XAttrNames_JSON_Schema)

3. For testing these operation, would you create an {{WebHdfsFileSystem}} 
instance and call {{getXAttr}}, {{getXAttrs}}, and {{listXAttrs}} as 
{{TestOfflineImageViewerForAcl}}?

> Implement GETXATTRS and LISTXATTRS operation for WebImageViewer
> ---------------------------------------------------------------
>
>                 Key: HDFS-8462
>                 URL: https://issues.apache.org/jira/browse/HDFS-8462
>             Project: Hadoop HDFS
>          Issue Type: New Feature
>            Reporter: Akira AJISAKA
>            Assignee: Jagadesh Kiran N
>         Attachments: HDFS-8462-00.patch, HDFS-8462-01.patch
>
>
> In Hadoop 2.7.0, WebImageViewer supports the following operations:
> * {{GETFILESTATUS}}
> * {{LISTSTATUS}}
> * {{GETACLSTATUS}}
> I'm thinking it would be better for administrators if {{GETXATTRS}} and 
> {{LISTXATTRS}} are supported.



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

Reply via email to