tchivs commented on PR #8208: URL: https://github.com/apache/paimon/pull/8208#issuecomment-4777411329
Thanks for the review. I pushed another follow-up commit: `0fe1ace2a`. Updates included in the latest branch: - Protected JDBC view operations with catalog locks consistently. `renameView` now locks source and target identifiers in deterministic order and performs source existence, target database, table/view collision checks, and metadata rename inside the lock. `alterView` now keeps the whole read-modify-write sequence inside one lock. - Replaced view conflict handling based on exception-message matching with a typed internal `JdbcViewConflictException` kind. - Restored the thread interrupted flag when view operations catch `InterruptedException`, including the shared JDBC execute path. - Included view-only databases in both `listDatabases` and `databaseExists` so `listViews` is consistent with the metadata returned by `listDatabases`. - Removed redundant view method overrides from `CachingCatalog`. Validation run locally: - `git diff --check` - `mvn -pl paimon-core spotless:check -DskipTests` - `mvn -pl paimon-core -am -Pfast-build -DfailIfNoTests=false -Dtest=JdbcCatalogTest test` -- 69 tests, 0 failures - `mvn -pl paimon-core -am -DfailIfNoTests=false -Dtest=JdbcCatalogTest test` -- 69 tests, 0 failures PTAL when you have time. -- 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]
