[jira] [Commented] (ARROW-14111) [C++] Add extraction function support for time32/time64

["Joris Van den Bossche (Jira)" <ji...@apache.org>]

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

Joris Van den Bossche commented on ARROW-14111:
-----------------------------------------------

> It's probably debatable if something like millisecond(time32('s')) should be supported but for ease of use it would probably be simplest to just hard code these to return 0.

It can be consistent with how this is handled for timestamp[s], and I think we indeed return 0 for those

> [C++] Add extraction function support for time32/time64
> -------------------------------------------------------
>
>                 Key: ARROW-14111
>                 URL: https://issues.apache.org/jira/browse/ARROW-14111
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Weston Pace
>            Assignee: Rok Mihevc
>            Priority: Major
>              Labels: kernel, query-engine
>
> time64('ns') should support:
>  - hour
>  - minute
>  - second
>  - millisecond
>  - microsecond
>  - nanosecond
>  - subsecond
> time64('us'), time32('ms'), and time32('s') should support those values that fit their resolution.
> It's probably debatable if something like millisecond(time32('s')) should be supported but for ease of use it would probably be simplest to just hard code these to return 0.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)