Repository navigation
[C++] Support duration type in floor_temporal/etc #50395
Description
Describe the enhancement requested
The temporal rounding kernels round_temporal, floor_temporal, and ceil_temporal are currently only registered for date32/64, time32/64, and timestamp inputs (WithDates, WithTimes, WithTimestamps in scalar_temporal_unary.cc).
It would be useful to round duration inputs to multiples of physical and perhaps some calendar units.
Component(s)
C++
Activity
docs(lazy/upstream): AG12 — Arrow temporal downcast floor gap (dup pa…
snigdhachoppac commentedon Jul 28, 2026
Hi, I’d like to work on this issue. Would it be acceptable to start by adding DurationType support for physical units to round_temporal, floor_temporal, and ceil_temporal, along with focused C++ tests, while leaving calendar-unit semantics out unless they are specifically desired?
rok commentedon Jul 28, 2026
Hey @snigdhachoppac. For calendar units only day and week are non-ambiguous enough to make sense to add them. Please go ahead and open the PR. Once you do ping me and I'll assign you this issue.
apacheGH-50395: [C++] Support duration inputs in temporal rounding
snigdhachoppac commentedon Jul 28, 2026
Thanks, @rok I’ve opened PR implementing duration support for physical units, including day and week, while leaving month, quarter, and year unsupported.
apacheGH-50395: [C++] Expand duration rounding test coverage
apacheGH-50395: [C++] Guard duration week multiple overflow
GH-50395: [C++] Support duration inputs in temporal rounding (#50675)
rok commentedon Jul 31, 2026
Issue resolved by pull request 50675
#50675