[
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)
