You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@streams.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2014/09/09 22:34:28 UTC

[jira] [Commented] (STREAMS-166) S3PersistReader does not correctly handle directory listing

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

ASF GitHub Bot commented on STREAMS-166:
----------------------------------------

GitHub user robdouglas opened a pull request:

    https://github.com/apache/incubator-streams/pull/80

    STREAMS-166 | Modified the S3PeristReader to check for object summaries ...

    ...in the listing response. If there are any object summaries then we can assume that the path is pointing to a directory and we need to iterate through all files that it contains.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/robdouglas/incubator-streams STREAMS-166

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-streams/pull/80.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #80
    
----
commit 8fb07eb2f3ebfae42f612ad169c06115914d8a4f
Author: Robert Douglas <rd...@w2odigital.com>
Date:   2014-09-09T20:32:50Z

    STREAMS-166 | Modified the S3PeristReader to check for object summaries in the listing response. If there are any object summaries then we can assume that the path is pointing to a directory and we need to iterate through all files that it contains.

----


> S3PersistReader does not correctly handle directory listing
> -----------------------------------------------------------
>
>                 Key: STREAMS-166
>                 URL: https://issues.apache.org/jira/browse/STREAMS-166
>             Project: Streams
>          Issue Type: Bug
>            Reporter: Robert Douglas
>   Original Estimate: 3h
>  Remaining Estimate: 3h
>
> In its current form, the S3PeristReader is not able to pick up on files inside of a given directory. In addition to checking for common prefixes, we need to ensure there are no object summaries in the listing response before we assume we are dealing with a single file.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)