zanmato1984 commented on issue #33962:
URL: https://github.com/apache/arrow/issues/33962#issuecomment-1864951920
Thanks @jorisvandenbossche for the informative comment!
Though I'm neutral to adding or not adding these kernels, I think the
following
> so it doesn't necessarily make sense to copy the interface of Python's
`datetime.timedelta` to extract those components (for example, why days and
seconds, and not also hours?). Also note that the Python attributes are plural,
in contrast to the names for the timestamp/date/time parts.
makes a great point. I felt the same (unintuitive) way when I was adding the
Python test case in my PR:
```
days = td.dt.days.astype("int64")
...
assert pc.day(tda).equals(pa.array(days))
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
