[ 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