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

Brahma Reddy Battula commented on HADOOP-16077:
-----------------------------------------------

{quote}bq. The extra RPC case shall come only in case of if -sp and -e 
parameter are used together. Else it shall be only one . And Introducing in the 
getContentSummary isn't a problem but thats part of Hadoop Common and usually 
we should prefer to refrain from introducing into the common segment just for 
pourpose of HDFS and moreover getContentSummary is quite an heavy operation as 
compared to getStoaragePolicy. For ec policy it was added in the content 
summary since we don't have getECpolicy in FileSystem it is only in HDFS.
{quote}
Already getContentSummary()  is called for EC policy, so it will not be 
additional. And ECPolicy added in contensummary for Policy summarization(see 
HDFS-11647).
{quote}The big enemy there is someone doing a "hadoop fs -ls -sp /" .
{quote}
I was talking about this recursive call only.

 

> 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, HADOOP-16077-05.patch, 
> HADOOP-16077-06.patch, HADOOP-16077-07.patch
>
>




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

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

Reply via email to