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

Surendra Singh Lilhore commented on HADOOP-16077:
-------------------------------------------------

Thanks [~ayushtkn] for patch.

One minor comment.

{{storagepolicies -getStoragePolicy}} command returns message "The storage 
policy of <path> is unspecified" when policy is not set, but after this change 
{{ls}} command will  return default policy in this case.
{code:java}
          if (storagePolicyId ==
              HdfsConstants.BLOCK_STORAGE_POLICY_ID_UNSPECIFIED) {
            System.out.println("The storage policy of " + path
                + " is unspecified");
            return 0;
          }
{code}
Better update the "ls" command document for this behaviour.

> Add an option in ls command to include storage policy
> -----------------------------------------------------
>
>                 Key: HADOOP-16077
>                 URL: https://issues.apache.org/jira/browse/HADOOP-16077
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: common
>    Affects Versions: 3.3.0
>            Reporter: Ayush Saxena
>            Assignee: Ayush Saxena
>            Priority: Major
>         Attachments: HADOOP-16077-01.patch, HADOOP-16077-02.patch, 
> HADOOP-16077-03.patch, HADOOP-16077-04.patch
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to