[
https://issues.apache.org/jira/browse/ARROW-1613?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16193237#comment-16193237
]
ASF GitHub Bot commented on ARROW-1613:
---------------------------------------
Github user BryanCutler commented on the issue:
https://github.com/apache/arrow/pull/1138
Thanks @icexelloss. I made it an alternative `close` because `ArrowWriter`
has an `end()` and the meaning there is that writing is done and it is
end-of-stream (for the stream format), but doesn't do anything with resources.
Here it is specifically to close resources so I think keeping the name `close`
makes more sense and the original `close()` is still there which acts the same
way.
> [Java] ArrowReader should not close the input ReadChannel
> ---------------------------------------------------------
>
> Key: ARROW-1613
> URL: https://issues.apache.org/jira/browse/ARROW-1613
> Project: Apache Arrow
> Issue Type: Improvement
> Components: Java - Vectors
> Reporter: Bryan Cutler
> Assignee: Bryan Cutler
> Priority: Minor
> Labels: pull-request-available
>
> Currently, {{ArrowReader.close()}} will close resources (VectorSchemaRoot and
> Dictionary Vectors) and also close the input ReadChannel, or InputStream for
> ArrowStreamReader. Closing of the ReadChannel should be done by what ever
> created it because it might need to be reused.
> If this not possible, an alternative could be to add a method
> {{ArrowReader.end()}} that will close resources but not the ReadChannel.
> Then {{end()}} could be called instead of {{close()}}.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
