SEZ9 commented on PR #12454: URL: https://github.com/apache/seatunnel/pull/12454#issuecomment-5992758474
## Module-level results on `ba304980c71f`, as promised The Build run on this head has finished: [run 37260524575](https://github.com/SEZ9/seatunnel/actions/runs/37260524575) — **94 jobs, zero failures**. Below is the per-module figure for each piece this PR still carries, so each kept change has a result on *this* head rather than on a superseded one. Citations are the Java 11 logs; the Java 8 matrix passed as well (no job in the run concluded anything other than `success` or `skipped`). | What it covers | Test class | Result | Job | |---|---|---|---| | PostgreSQL CDC LSN boundary | `PostgresCDCIT` | **37 tests, 0 failures, 0 errors, 0 skips** (968.8 s) | [all-connectors-it-7 (11)](https://github.com/SEZ9/seatunnel/actions/runs/37260524575/job/111607191973), 04:59:22 UTC | | Recovery hang fix | `CheckpointCoordinatorFailoverIT` | 4 tests, 0 failures, 0 errors, 0 skips (94.3 s) | [engine-v2-it (11)](https://github.com/SEZ9/seatunnel/actions/runs/37260524575/job/111607192122), 05:55:19 UTC | | `parallelism = 1` hunk (#12216) | `TestFilterRowKindIT` | 14 tests, 0 failures, 0 errors, 0 skips (588.3 s) | [transform-v2-it-part-1 (11)](https://github.com/SEZ9/seatunnel/actions/runs/37260524575/job/111607192211), 06:10:49 UTC | | PayPal 503 OAuth e2e coverage | `PayPalIT` | 7 tests, 0 failures, 0 errors, 0 skips (168.7 s) | [all-connectors-it-6 (11)](https://github.com/SEZ9/seatunnel/actions/runs/37260524575/job/111607192133), 08:21:25 UTC | @DanielLeens this is the figure I said I would post rather than reuse — the 37/0/0/0 I quoted on 2026-09-27 was measured on `4f86cb0ac63e`, and the one above is measured on `ba304980c71f`. The number happens to be identical, which is the expected outcome given the rebase touched no PostgreSQL file, but it is now an independent measurement rather than a carried-over one. ### One thing in the run that looks like a gap and is not Eleven jobs in that run are `skipped`, including the whole `updated-modules-integration-test-part-1..8` family. That is the gating in `backend.yml`, not dropped coverage: ```yaml updated-modules-integration-test-part-1: if: needs.changes.outputs.api == 'false' && needs.changes.outputs.engine == 'false' && needs.changes.outputs.it-modules != '[]' ``` ```yaml all-connectors-it-1: if: needs.changes.outputs.api == 'true' || needs.changes.outputs.engine == 'true' ``` The two are mutually exclusive by construction. Because this PR touches the engine, the narrow changed-modules path is skipped and `all-connectors-it-1..7` runs the whole of `seatunnel-connector-v2-e2e` instead — a superset of what the skipped jobs would have covered. That is also why `PostgresCDCIT` and `PayPalIT` above come out of `all-connectors-it-*` rather than out of a per-connector job. ### Still outstanding from my side Nothing. `ba304980c71f` is the head I intend to be final, I have no further pushes planned, and I will not push again without saying so here first. The only open item is review of this head — the earlier approval of `3080371ce733` no longer applies after the cleanup push, as noted above. I am also not re-raising the stress numbers: I retracted that commitment above and the retraction stands. -- 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]
