You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2023/05/04 14:57:21 UTC

[GitHub] [arrow-datafusion] alamb commented on a diff in pull request #6230: Alamb/better error message

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


##########
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 would like to hear opinions on this new message. I think it is better but still not great as it has references to the detailed Arrow types.
   
   



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