hameizi opened a new pull request #15638: URL: https://github.com/apache/flink/pull/15638
### What is the purpose of the change There are some scenes we can't drop table by hive catalog because of we execute error create table sql like "caeate table tableName". And then when we execute drop table sql "drop table tableName" will throw error "org.apache.flink.table.catalog.exceptions.CatalogException: Failed to get table schema from properties for generic table". So, i think maybe we should remove the check when we drop table by third-party catalog and just drop table depend on the third-party catalog. I think we should give the catalog more freedom to drop table, to let user can drop because of they execute error create table sql. ### Brief change log Some scenes can't drop table by hive catalog ### Verifying this change Manually verified. ### Does this pull request potentially affect one of the following parts: Dependencies (does it add or upgrade a dependency): (yes / **no**) The public API, i.e., is any changed class annotated with @Public(Evolving): (**yes** / no) The serializers: (yes / **no** / don't know) The runtime per-record code paths (performance sensitive): (yes / **no** / don't know) Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / **no** / don't know) The S3 file system connector: (yes / **no** / don't know) ### Documentation Does this pull request introduce a new feature? (yes / **no**) If yes, how is the feature documented? (not applicable / docs / JavaDocs / **not documented**) -- 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: [email protected]
