DanielLeens commented on PR #12895: URL: https://github.com/apache/seatunnel/pull/12895#issuecomment-6338006909
A CI-status correction on unchanged head `6ce59eb6b64461055cba4df541648df81ea63a0a`: my previous note described a queued/pending Build. The current [Build](https://github.com/apache/seatunnel/runs/119429474407) is now **failure**, so the old pending guidance is obsolete. Failed job families: `Run / unit-test`, `Run / engine-v2-it`. Concrete inspected log evidence (one representative JDK/OS per failed family; this does not claim every matrix leg has the identical cause): - [Run / unit-test (11, ubuntu-latest)](https://github.com/nzw921rx/seatunnel/actions/runs/39605390741/job/119279392901): `LLMBooleanOutputTest.readsLegacyFactoryWrapperWithValidationDisabled` fails with `InvalidClassException` on `AbstractMultiCatalogTransform`. - [Run / engine-v2-it (11, ubuntu-latest)](https://github.com/nzw921rx/seatunnel/actions/runs/39605390741/job/119279394082): failed case(s): `testCheckpointsKeepCompletingUnderSustainedBackpressure`. Upstream/next-step check: The archived LLM fixture is tracked in #12929; proposed fix #12938 remains open. Latest `dev` (`99ea16b9d6bdce452d4c4955581ccc7236a81bbd`) still has no explicit historical UID on `AbstractMultiCatalogTransform`. Syncing today will not fix that observed failure. Once the fix lands in `dev`, sync and rerun the affected failed jobs. Keep unrelated transform-serialization changes out of this PR. For `BackpressureSlowSinkIT`, the deterministic-test proposal #12313 and source-lock handoff #12316 are still open. No corresponding merged cure was found in the current base. Treat that lane separately from the unit-test fixture issue; retry only the failed lane when appropriate, and keep its new result if the assertion recurs. My previous source-level conclusion remains unchanged: no outstanding source blocker from my side. This is a CI-status follow-up, not a fresh full code review or formal approval. The branch is ahead 3, behind 8, compare `diverged` against `dev`; this drift is recorded independently, not used as a generic sync request. A successful required Build and write-capable maintainer approval remain merge gates. Thanks for the contribution and for keeping the fix 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]
