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

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

BryanCutler commented on issue #1371: ARROW-1866: [Java] Combine MapVector 
classes and remove NonNullableMapVector
URL: https://github.com/apache/arrow/pull/1371#issuecomment-349052439
 
 
   Thanks for the details @jacques-n, I am fine with keeping the 2 level 
hierarchy for now and it seems like we need some more discussion to move 
forward with `StructType` naming.  I'll proceed with option (3) and restore the 
previous `MapVector` class names. 

----------------------------------------------------------------
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:
[email protected]


> [Java] Combine MapVector and NonNullableMapVector Classes
> ---------------------------------------------------------
>
>                 Key: ARROW-1866
>                 URL: https://issues.apache.org/jira/browse/ARROW-1866
>             Project: Apache Arrow
>          Issue Type: Sub-task
>          Components: Java - Vectors
>            Reporter: Bryan Cutler
>            Assignee: Bryan Cutler
>              Labels: pull-request-available
>             Fix For: 0.8.0
>
>
> {{NonNullableMapVector}} class can be merged into {{MapVector}} and removed 
> as part of removing the non nullable vectors.



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

Reply via email to