You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Genmao Yu (JIRA)" <ji...@apache.org> on 2017/11/13 08:13:00 UTC

[jira] [Comment Edited] (HADOOP-14993) AliyunOSS: Override listFiles and listLocatedStatus

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

Genmao Yu edited comment on HADOOP-14993 at 11/13/17 8:12 AM:
--------------------------------------------------------------

[~drankye] I post a new patch, some updates related to your comments:
1. refactor {{listFiles}} and {{listLocatedStatus}}
2. use {{firstListing}} to avoid the {{null}} check of {{batchIterator}}, so I kept it.
3.& 4. IMHO it is ok for current naming, copy from existing code.

All test passed against "oss-cn-shanghai.aliyuncs.com".

cc [~stevel@apache.org] please take a review, thanks.


was (Author: unclegen):
[~drankye] I post a new patch, some updates related to your comments:
1. refactor {{listFiles}} and {{listLocatedStatus}}
2. use {{firstListing}} to avoid the {{null}} check of {{batchIterator}}, so I kept it.
3.& 4. IMHO it is ok for current naming, copy from existing code.

also cc [~stevel@apache.org] please take a review, thanks.

> AliyunOSS: Override listFiles and listLocatedStatus 
> ----------------------------------------------------
>
>                 Key: HADOOP-14993
>                 URL: https://issues.apache.org/jira/browse/HADOOP-14993
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/oss
>    Affects Versions: 3.0.0-beta1
>            Reporter: Genmao Yu
>            Assignee: Genmao Yu
>         Attachments: HADOOP-14993.001.patch, HADOOP-14993.002.patch, HADOOP-14993.003.patch
>
>
> Do a bulk listing off all entries under a path in one single operation, there is no need to recursively walk the directory tree.
> Updates:
> - override listFiles and listLocatedStatus by using bulk listing
> - some minor updates in hadoop-aliyun index.md



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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