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

Ayush Saxena commented on HADOOP-16077:
---------------------------------------

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.

So IMO that is generally not required, but if you insist on it I can extend, As 
far as implementation is concerned that isn't that big , won't take any 
considerable effort.

This case ,but will  only come up only both ec and storage are to be fetched 
together else both operations run fine.

But we need to be sure that doesn't effect object stores and all and doesn't 
have much difference. because I wouldn't want to unnecessary problems to them 
too.

An extra opinions won't hurt. [~surendrasingh] and [~ste...@apache.org] have 
gone through the initial patches. Can you help in the context. If all have 
similar thinking, I would extend the way suggested

[~ste...@apache.org] if patch needs to be changed. Would require your help 
since this code is already in trunk. I guess got committed with your previous 
commit.



> 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