Peng-Lei opened a new pull request #33339: URL: https://github.com/apache/spark/pull/33339
### What changes were proposed in this pull request? Add the name check when initialize a catalog by name. Keep consistent with the namespace/table naming rule. ### Why are the changes needed? When I try to add the command `SHOW CATALOGS`, I found catalog names are meaningless in some cases. eg: 1. if the catalog name is `""`, `" "`, `" "` that contains some spaces and so on. I think it doesn't make sense 2. if the catalog name is `"a.b"` that contains `.`, I'm more confused. `b` is the name of a namespace or part of catalog name? ### Does this PR introduce _any_ user-facing change? Yes. If user want to regist external catalog plugin. user shoule keep catalog name in SparkConf is compliant. ### How was this patch tested? Add ut test -- 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]
