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

Li Jin commented on ARROW-1816:
-------------------------------

I will look at cell level a bit closer, my hunch is branching can be avoided by 
storing a timeUnit object in the vector:
https://github.com/apache/arrow/blob/master/java/vector/src/main/java/org/apache/arrow/vector/NullableTimeStampNanoVector.java#L116

timeZone doesn't seem to be used in cell level either, but I need to look 
closer.

> [Java] Resolve new vector classes structure for timestamp, date and maybe 
> interval
> ----------------------------------------------------------------------------------
>
>                 Key: ARROW-1816
>                 URL: https://issues.apache.org/jira/browse/ARROW-1816
>             Project: Apache Arrow
>          Issue Type: Sub-task
>            Reporter: Li Jin
>             Fix For: 0.8.0
>
>
> Personally I think having 8 vector classes for timestamps is not great. This 
> is discussed at some point during the PR:
> https://github.com/apache/arrow/pull/1203#discussion_r145241388



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

Reply via email to