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

Steve Loughran commented on HADOOP-13321:
-----------------------------------------

the marking up looks good. Could you do the same for {{FTPFileSystem.}} and 
{{ChecksumFileSystem}}? Once that's done I'll be +1

I looked at one fo the tests {{TestFsShellCopy}} to see if we should be 
ignoring the tests there but no, some of the test are doing exactly what we 
don't want: exists, isFile, listFileStatus calls...marking them as deprecated 
shows up where we have work to do in future.

> Deprecate FileSystem APIs that promote inefficient call patterns.
> -----------------------------------------------------------------
>
>                 Key: HADOOP-13321
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13321
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>    Affects Versions: 3.0.0-alpha3
>            Reporter: Chris Nauroth
>            Assignee: Mingliang Liu
>         Attachments: HADOOP-13321.000.patch, HADOOP-13321.001.patch, 
> HADOOP-13321.002.patch, HADOOP-13321.003.patch
>
>
> {{FileSystem}} contains several methods that act as convenience wrappers over 
> calling {{getFileStatus}} and retrieving a single property of the returned 
> {{FileStatus}}.  These methods have a habit of fostering inefficient call 
> patterns in applications, resulting in multiple redundant {{getFileStatus}} 
> calls.  For HDFS, this translates into wasteful NameNode RPC traffic.  For 
> file systems backed by cloud object stores, this translates into wasteful 
> HTTP traffic.  This issue proposes to deprecate these methods and instead 
> encourage applications to call {{getFileStatus}} and then reuse the same 
> {{FileStatus}} instance as needed.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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

Reply via email to