[jira] [Commented] (ARROW-1962) [Java] Add reset() to ValueVector interface

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on ARROW-1962:
---------------------------------------

BryanCutler opened a new pull request #1455: [WIP] ARROW-1962: [Java] Adding reset to ValueVector interface
URL: https://github.com/apache/arrow/pull/1455
 
 
   Adding `reset()` to the ValueVector interface and implementing where it is not done already.  Removing unused abstract class BaseDataValueVector that is not used anymore by the UnionVector.
   
   WIP, should add some tests for reset if not there already.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> [Java] Add reset() to ValueVector interface
> -------------------------------------------
>
>                 Key: ARROW-1962
>                 URL: https://issues.apache.org/jira/browse/ARROW-1962
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Java - Vectors
>            Reporter: Bryan Cutler
>            Assignee: Bryan Cutler
>              Labels: pull-request-available
>
> The {{reset()}} method exists in some ValueVectors but not all.  Its meaning is that it will bring the vector to an empty state, but not release any buffers (as opposed to clear() which resets and releases buffers).
> It should be added to the {{ValueVector}} interface and implemented in the vector hierarchy where it currently is not. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)