srielau commented on code in PR #53530:
URL: https://github.com/apache/spark/pull/53530#discussion_r2694921040
##########
common/utils/src/main/resources/error/error-conditions.json:
##########
@@ -1037,6 +1037,48 @@
},
"sqlState" : "21S01"
},
+ "CURSOR_ALREADY_EXISTS" : {
Review Comment:
I'm not a huge fan of subclasses. One of my major beefs is that it collapses
the SQLSTATEs and we end up with inappropriate states for part of thr
subclasses.
CURSOR_ALREADY_EXISTS collapses into "object already exists" (42XXX), an
analysis failure. While cursor state errors fall under 24XXX which is a SQL
Standard state specifically for those, and it's runtime.
--
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]
