Peng-Lei commented on pull request #33339:
URL: https://github.com/apache/spark/pull/33339#issuecomment-909008568
do a sumary:
1. shall we do a validation of catalog name?
I think it is necessary，the reason is as description of PR
[#description](https://github.com/apache/spark/pull/33339#issue-689616767)
2. do we keep the consistence with namespace/table or just empty check?
just do empty check is simple, but it is incomplete. I considered
catalog as database object, so keep the consistence with namespace/table
3. where is validation locate? `SparkConf` update or get a catalog throught
`catalog(name)` or load a catalog `Catalogs.load` ?
Could you give me some advices? @cloud-fan @viirya @sunchao @HyukjinKwon
@dongjoon-hyun
--
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]
