nielifeng commented on PR #11434: URL: https://github.com/apache/seatunnel/pull/11434#issuecomment-4987608915
Thanks @DanielLeens. The current head (`e5ddd4f06`) does not contain any JDBC catalog changes. The diff is scoped to the Flink/Spark config-validation feature only — all 17 changed files live under `seatunnel-core`: - `seatunnel-core-starter`: `ConfigValidationUtils` + its test, 4 test-config JSONs, pom - `seatunnel-flink-starter-common`: `FlinkConfValidateCommand` + test, 2 test configs, pom - `seatunnel-spark-starter-common`: `SparkConfValidateCommand` + test, 2 test configs, pom There is no `PostgresCatalog.java`, `JdbcPostgresIT.java`, or `JdbcOpenGaussIT.java` in this branch (the only `catalog` references are the core `org.apache.seatunnel.api.table.catalog.CatalogTable` API used to load source/transform/sink tables during validation). The classloader / `MetadataConfig` points from the Copilot review were addressed in `3646fc626`. The earlier `Build` failure was an unrelated flaky infra issue (the Databend E2E testcontainer failed to start with a connection reset — this PR does not touch any connector). I've re-triggered the failed jobs. Could you re-review once CI is green? Thanks! -- 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]
