berkaysynnada opened a new issue, #5413:
URL: https://github.com/apache/arrow-datafusion/issues/5413
**Is your feature request related to a problem or challenge? Please describe
what you are trying to do.**
While we can add intervals to timestamp variants (LHS: Timestamp (like
`TimestampSecond`), RHS: Interval (like `IntervalYearMonth`)), we cannot
replace the sides (LHS: Interval, RHS: Timestamp), and get the error
`Err(Internal("Operator + is not implemented for variants
IntervalYearMonth(\"1\") and TimestampSecond(12345678, None)"))`
Addition should be commutative for intervals and timestamps, unlike
subtraction.
**Describe the solution you'd like**
In `impl_op` macro, the match arms can be extended to handle Interval
variants for addition, and an adder function can be implemented.
**Describe alternatives you've considered**
Only the order of the parameters can be changed before calling that
`impl_op` macro since **Timestamp + Interval** operations are working.
**Additional context**
This feature can also be extended to cover operations between interval
variants (`IntervalYearMonth`, `IntervalDayTime`, `IntervalMonthDayNano`). We
can use addition and subtraction operators between them (negatively resulting
situations should be under consideration).
--
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: github-unsubscr...@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
