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

Wes McKinney commented on ARROW-316:
------------------------------------

[~julienledem] I guess we should revise the description to be "UNIX time", 
which does not include leap seconds (https://en.wikipedia.org/wiki/Unix_time).

One option is to add an enum like:

{code}
enum DateUnit: short { 
  DAY,   /// 32-bit storage
  MILLISECOND,   /// 64-bit storage
}
{code}

and add that to the Date flatbuffer table. 

> Finalize Date type
> ------------------
>
>                 Key: ARROW-316
>                 URL: https://issues.apache.org/jira/browse/ARROW-316
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: Format
>            Reporter: Julien Le Dem
>            Assignee: Julien Le Dem
>             Fix For: 0.3.0
>
>
> Parquet defines it as: "number of days from the Unix epoch, 1 January 1970."
> https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#datetime-types
> We should make it the same.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to