mchades opened a new pull request, #12798: URL: https://github.com/apache/gravitino/pull/12798
### What changes were proposed in this pull request? Extend existing catalog connection testing to accept proposed `CatalogChange` values. The changes are applied to a temporary effective catalog configuration before running the existing connection probe. The temporary catalog is always closed, and no catalog configuration or secret material is persisted. This also adds REST, OpenAPI, Java client, Python client, and regression test coverage. ### Why are the changes needed? The existing API can only test the stored catalog configuration. Users need to validate proposed catalog changes before altering the catalog. Fix: #12794 ### Does this PR introduce _any_ user-facing change? Yes. - Adds `testConnection(String, CatalogChange...)` for existing catalogs. - Allows an optional `CatalogUpdatesRequest` body on the existing-catalog connection-test endpoint. Omitting the body retains the existing stored-configuration behavior. - Adds corresponding Java and Python client support. - No property keys are added or removed. ### How was this patch tested? - Focused Core, REST, and Java client unit tests. - Hive Docker integration test covering temporary configuration and non-persistence. - Python client unit tests, Black, and Ruff. - Spotless checks. - `./gradlew :docs:build :api:javadoc -PskipITs`. - `git diff --check`. -- 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]
