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

Julien Le Dem commented on ARROW-617:
-------------------------------------

 [~wesmckinn] that would work but the type needs to fully specify the physical 
layout. If the unit is used to infer the value bit width (32 or 64 bits) then 
we can't change in the future. So the type would be something like this?
```
table Time {
  unit: TimeUnit;
  bitWidth: int; // restricted to 32 for sec and millis, and 64 for micros and 
nanos in v1
}
```
Possibly we can further limit to millis and nanos only in v1 since they are 
respectively the most precise for each bitWIdth (millis: 32bits, nanos: 64 bits)
This is a little verbose but explicit and future proof.

> Time type is not specified clearly
> ----------------------------------
>
>                 Key: ARROW-617
>                 URL: https://issues.apache.org/jira/browse/ARROW-617
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: Format
>            Reporter: Julien Le Dem
>
> 2 options:
> - Use 64 bits for microseconds and nanoseconds, 32 bits for other units
> - Use 64 bits for everything
> The latter is simpler to implement, the former saves space.



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

Reply via email to