xuefuz commented on issue #8390: [FLINK-12469][table] Clean up catalog API on default/current database URL: https://github.com/apache/flink/pull/8390#issuecomment-491088370 > Thanks @xuefuz for the PR. > > Currently this PR doesn't seem to verify whether user-set "defaultDatabase" in catalog impls' constructors actually exists or not upon initialization. Do we need to ensure it actually exists? if so, how? I have something in mind but want to listen to your ideas first. default DB is most likely from configuration and it cannot be verified until a catalog is opened. This shouldn't be a problem as error will be reported later when the db is actually referenced.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
