diqiu50 opened a new pull request, #11349: URL: https://github.com/apache/gravitino/pull/11349
### What changes were proposed in this pull request? - Fix `GravitinoPaimonCatalog.dropTable` to fall back to `asViewCatalog().dropView()` when `purgeTable` returns false, so `DROP VIEW` works correctly on Paimon catalogs. - Move the 9 view integration tests from `FlinkHiveCatalogIT` into the shared `FlinkCommonIT` base class, controlled by a `supportViewOperation()` hook and a `baseTableConnectorClause()` helper. - Enable view tests for Iceberg (Hive + REST backends) and Paimon (Hive + JDBC backends). ### Why are the changes needed? The Flink connector only supported and tested views for the Hive catalog. Iceberg and Paimon catalogs already implement `asViewCatalog()` on the server side but had no Flink-level test coverage, and `GravitinoPaimonCatalog.dropTable` had a bug that prevented `DROP VIEW` from working. Fix: #11348 ### Does this PR introduce _any_ user-facing change? Yes. `CREATE/DROP/ALTER/SHOW VIEW` now work in Flink SQL when using an Iceberg or Paimon catalog backed by Gravitino. ### How was this patch tested? Added 9 view integration tests (create, list, drop, rename, replace body, query, if-not-exists variants, table/view list separation) to `FlinkCommonIT`. Tests run against Iceberg Hive/REST backends and Paimon Hive/JDBC backends via Docker integration tests. -- 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]
