DanielLeens commented on PR #10962: URL: https://github.com/apache/seatunnel/pull/10962#issuecomment-6338114635
A CI-status correction on unchanged head `a17ee406e0fe28b83c33908370b171f1efaed9f7`: my previous note described a queued/pending Build. The current [Build](https://github.com/apache/seatunnel/runs/118591367955) is now **failure**, so the old pending guidance is obsolete. Failed job families: `Run / unit-test`, `Run / engine-v2-it`, `Run / Dependency licenses`, `Run / kudu-connector-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/aakamara23/seatunnel/actions/runs/39392977060/job/118529742847): `LLMBooleanOutputTest.readsLegacyFactoryWrapperWithValidationDisabled` fails with `InvalidClassException` on `AbstractMultiCatalogTransform`. - [Run / engine-v2-it (11, ubuntu-latest)](https://github.com/aakamara23/seatunnel/actions/runs/39392977060/job/118529742912): failed case(s): `testCheckpointsKeepCompletingUnderSustainedBackpressure`. - [Run / Dependency licenses](https://github.com/aakamara23/seatunnel/actions/runs/39392977060/job/118529743152): `tools/dependencies/checkLicense.sh` rejects a packaged-JAR inventory difference, `+jna-5.13.0.jar`. - [Run / kudu-connector-it (11, ubuntu-latest)](https://github.com/aakamara23/seatunnel/actions/runs/39392977060/job/118529743916): Maven wrapper bootstrap fails before the connector tests (`MavenWrapperMain` is missing). 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. The missing Maven wrapper class is a bootstrap failure, not an executed connector assertion. No matching base-branch wrapper correction was identified. A targeted retry of that failed job after download/bootstrap availability recovers is the useful diagnostic; do not add product dependencies to compensate. The dependency-license lane is an additional gate, not explained by the LLM fixture. Please account for the newly packaged `jna-5.13.0.jar` in the dependency/shading and license inventory review already requested for this feature. Keep the license checker enabled; do not assume an upstream sync will reconcile a new distribution artifact. My earlier source/test/documentation blockers remain in force; these CI facts do not clear them. Please follow the existing numbered findings before requesting a new full review. The branch is ahead 10, behind 10, 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]
