DanielLeens commented on PR #12065: URL: https://github.com/apache/seatunnel/pull/12065#issuecomment-5539308884
Quick CI update since my last review only knew the Build run was still in progress: it has since finished, and it failed — but on a job that has nothing to do with this PR's diff. The failing job is `Run / transform-v2-it-part-1 (8, ubuntu-latest)`, specifically `TestFilterRowKindIT.testFilterRowKindMultiTable` on the Spark 2/3 engines (`expected: <0> but was: <1>` on a row-count assertion). This PR only touches `seatunnel-api`'s reusable config-validation-result plumbing — it never goes near the FilterRowKind transform or the Spark multi-table pipeline this test exercises, so this looks like a pre-existing, environment-timing-sensitive E2E flake rather than something introduced by this change. All of the module-relevant jobs (Code style, all four unit-test matrix combinations) are green. Since this isn't tied to your diff, I don't think there's anything to change in the PR itself — a CI rerun on the failing job should be enough to get a clean signal. My "Ready to merge" conclusion from the last review stands either way. -- 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]
