cloud-fan commented on code in PR #42917:
URL: https://github.com/apache/spark/pull/42917#discussion_r1329519381
##########
common/utils/src/main/resources/error/error-classes.json:
##########
@@ -860,6 +860,50 @@
"Exceeds char/varchar type length limitation: <limit>."
]
},
+ "EXPECT_PERMANENT_VIEW_NOT_TEMP" : {
+ "message" : [
+ "'<operation>' expects a permanent view but <viewName> is a temp view."
+ ]
+ },
+ "EXPECT_TABLE_NOT_VIEW" : {
+ "message" : [
+ "'<operation>' expects a table but <viewName> is a view."
+ ],
+ "subClass" : {
+ "NO_ALTERNATIVE" : {
+ "message" : [
+ ""
+ ]
+ },
+ "USE_ALTER_VIEW" : {
+ "message" : [
+ "Please use ALTER VIEW instead."
+ ]
+ }
+ }
+ },
+ "EXPECT_TABLE_OR_PERMANENT_VIEW_NOT_TEMP" : {
Review Comment:
Actually, seems we can reuse `EXPECT_PERMANENT_VIEW_NOT_TEMP`? If we get a
temp view, it's probably good enough to notify users that the operation needs
permanent view not temp, no need to mention that the operation accepts table as
well.
--
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]
