crunch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gabriel Reid (JIRA)" <>
Subject [jira] [Resolved] (CRUNCH-567) close() triggers NPE if initialize() fails in 2 classes
Date Sat, 03 Oct 2015 15:03:27 GMT


Gabriel Reid resolved CRUNCH-567.
       Resolution: Fixed
         Assignee: Gabriel Reid  (was: Josh Wills)
    Fix Version/s: 0.14.0

Committed to master. 

Thanks for the patch [~srowen] (and BTW, yes we're still generally working with patches here).

> close() triggers NPE if initialize() fails in 2 classes
> -------------------------------------------------------
>                 Key: CRUNCH-567
>                 URL:
>             Project: Crunch
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 0.12.0
>            Reporter: Sean Owen
>            Assignee: Gabriel Reid
>            Priority: Trivial
>             Fix For: 0.14.0
>         Attachments: CRUNCH-567.patch
> Quite trivial, but just noticed this on a customer project: in AvroRecordReader and HFileInputFormat,
close() closes a reader object. However it will be null if initialize() is never called or
fails to complete normally, generating an NPE. It ends up being an issue since their close()
is called in some kind of finally block at some point.
> Attaching a patch for now but do we do pull requests here now instead?

This message was sent by Atlassian JIRA

View raw message