[GitHub] [arrow-datafusion] ozankabak commented on a diff in pull request #6230: Minor: improve error message for interval subtraction

["ozankabak (via GitHub)" <gi...@apache.org>]

ozankabak commented on code in PR #6230:
URL: https://github.com/apache/arrow-datafusion/pull/6230#discussion_r1186388479


##########
datafusion/core/tests/sqllogictests/test_files/timestamps.slt:
##########
@@ -1031,7 +1031,7 @@ SELECT '2000-01-01T00:00:00'::timestamp - '2010-01-01T00:00:00'::timestamp;
 0 years 0 mons -3653 days 0 hours 0 mins 0.000000000 secs
 
 # Interval - Timestamp => error
-statement error DataFusion error: Error during planning: interval can't subtract timestamp/date
+statement error DataFusion error: Error during planning: Invalid interval subtraction: Interval\(MonthDayNano\) \- Timestamp\(Nanosecond, None\)

Review Comment:
   I agree it is better than status quo.



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org