[
https://issues.apache.org/jira/browse/PARQUET-145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14240280#comment-14240280
]
Ryan Blue commented on PARQUET-145:
-----------------------------------
[~whoschek], could you submit that PR for
https://github.com/apache/incubator-parquet-mr? We no longer use the old
repository and I can't merge from it. Thanks!
> InternalParquetRecordReader.close() should not throw an exception if
> initialization has failed
> ----------------------------------------------------------------------------------------------
>
> Key: PARQUET-145
> URL: https://issues.apache.org/jira/browse/PARQUET-145
> Project: Parquet
> Issue Type: Bug
> Components: parquet-mr
> Affects Versions: parquet-mr_1.6.0
> Reporter: wolfgang hoschek
>
> InternalParquetRecordReader.close() currently throws a NullPointerException
> if initialize() didn't succeed and hence the "reader" remains null.
> This NPE happens with client usage idioms like this:
> {code}
> AvroParquetReader reader = ...
> try {
> // read some parquet data
> } finally {
> if (reader != null) { reader.close(); }
> }
> {code}
> I believe close() should read as follows:
> {code}
> public void close() throws IOException {
> if (reader != null) {
> reader.close();
> }
> }
> {code}
> Thoughts?
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
