crunch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gabriel Reid (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (CRUNCH-428) Disallow empty list of paths for file inputs
Date Wed, 25 Jun 2014 08:12:24 GMT

     [ https://issues.apache.org/jira/browse/CRUNCH-428?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Gabriel Reid updated CRUNCH-428:
--------------------------------

    Attachment: CRUNCH-428.patch

Here's the patch. I'm going to commit this right away because it's a simple bug and (as far
as I can see) there's nothing that could get (more) broken by this change.

> Disallow empty list of paths for file inputs
> --------------------------------------------
>
>                 Key: CRUNCH-428
>                 URL: https://issues.apache.org/jira/browse/CRUNCH-428
>             Project: Crunch
>          Issue Type: Bug
>    Affects Versions: 0.10.0, 0.8.3
>            Reporter: Gabriel Reid
>            Assignee: Gabriel Reid
>         Attachments: CRUNCH-428.patch
>
>
> There is currently no checking on the number of input paths supplied in a list to FileSourceImpl
and its subclasses. This can lead to a confusing error when the empty list is deserialized
from CrunchInputs, as an attempt is made to create a Path from an empty string.
> The various paths that construct FileSourceImpl subclasses should defensively protect
against creating a file source with an empty list of paths.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message