cloud-fan commented on code in PR #38490: URL: https://github.com/apache/spark/pull/38490#discussion_r1015144581
########## core/src/main/resources/error/error-classes.json: ########## @@ -668,6 +668,23 @@ } } }, + "LOCATION_ALREADY_EXISTS" : { + "message" : [ + "Cannot create the location <location> because it already exists." Review Comment: do we really need sub error classes here? I think we can do ``` Cannot name the managed table as <identifier>, as its associated location <location> already exists. Please pick a different table name, or remove the existing location first. ``` -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org