DanielLeens commented on PR #12893: URL: https://github.com/apache/seatunnel/pull/12893#issuecomment-6338269631
Thanks for the contribution. A CI-status correction on unchanged head `fb99154ab93093c36d47fae02c0a8615cd522a9e`: the [fork Build](https://github.com/ricky2129/seatunnel/actions/runs/39551655438) has now completed with **failure**, so my earlier description of an in-progress run is obsolete. Both Kafka integration lanes have completed successfully: [Run / kafka-connector-it (17, ubuntu-latest)](https://github.com/ricky2129/seatunnel/actions/runs/39551655438/job/119064028309), [Run / kafka-connector-it (11, ubuntu-latest)](https://github.com/ricky2129/seatunnel/actions/runs/39551655438/job/119064028410). The previously pending Kafka integration gate is therefore cleared. My previous conclusion remains that there is no source-level blocker from my side; the documented suggestions remain non-blocking. The four failed `unit-test` legs remain separate from that result. The inspected JDK 11 log fails `LLMBooleanOutputTest.readsLegacyFactoryWrapperWithValidationDisabled` with `InvalidClassException` on `AbstractMultiCatalogTransform`. The upstream fixture issue is now tracked by #12929 and proposed fix #12938, which is still open. Latest `dev` is still `99ea16b9d6bdce452d4c4955581ccc7236a81bbd`; syncing it today does not incorporate that fix. Once the correction lands in the base, sync and rerun the affected failed jobs. Please keep unrelated transform-serialization patches out of this PR. The base comparison is `diverged` against `dev` (ahead 3, behind 9); this drift is recorded separately. A successful required Build and final write-capable maintainer approval remain necessary. This comment updates CI facts only and is not a fresh full review or an approval. -- 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]
