MaxGekk commented on a change in pull request #35787:
URL: https://github.com/apache/spark/pull/35787#discussion_r823371845
##########
File path: core/src/main/resources/error/error-classes.json
##########
@@ -25,6 +25,10 @@
"CONCURRENT_QUERY" : {
"message" : [ "Another instance of this query was just started by a
concurrent session." ]
},
+ "DATETIME_OVERFLOW" : {
+ "message" : [ "The '%s' function overflows the input '%s' timestamp by %s
%s." ],
Review comment:
I don't think making errors too general is right approach. Error
messages should help users to identify and fix issues in their queries.
> I think we can omit the input details.
Could you explain how users should find the problematic row if we don't
provide any context as you propose?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
