You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/09/01 09:45:25 UTC

[GitHub] [spark] sunchao commented on pull request #33339: [SPARK-36133][SQL] Add empty check for catalog name

sunchao commented on pull request #33339:
URL: https://github.com/apache/spark/pull/33339#issuecomment-909410791


   IMO the catalog naming convention can be enforced from Spark side. Even though I feel it doesn't necessarily need to follow the V1 database/table naming which is inherited from Hive, it should be OK so as to keep it simple and less error prone. 


-- 
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-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org