DanielLeens commented on PR #12808: URL: https://github.com/apache/seatunnel/pull/12808#issuecomment-6337996237
Thanks for the ping, @ricky2129. The current head remains `2d1291e74dae36f1eafb6b5fd04c57134b695bf2`, so my earlier source-level conclusion is unchanged and I am not issuing another full review on the same code. The old pending-CI note is now obsolete: [Build](https://github.com/ricky2129/seatunnel/actions/runs/39551584003) has completed with failure. The four unit-test legs and both edge-agent legs are red. The inspected JDK 11 Linux logs in both families fail in the unrelated archived LLM fixture, `LLMBooleanOutputTest.readsLegacyFactoryWrapperWithValidationDisabled`, with `InvalidClassException` on `AbstractMultiCatalogTransform`. Tracking issue #12929 and proposed fix #12938 are still open; latest `dev` does not contain the historical-UID correction. Please wait for that shared fix, then sync and rerun the affected failed jobs for the updated head. A sync today alone will not clear this failure, and no transform-serialization patch belongs in this JDBC batch-reduction PR. A green required Build and a write-capable maintainer's final approval remain the gates. Thanks for keeping the contribution focused. -- 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]
