DanielLeens commented on PR #12958: URL: https://github.com/apache/seatunnel/pull/12958#issuecomment-6397568929
Thanks for the contribution. The CI fact in my previous review has changed: the exact-head [fork Build](https://github.com/Nikk8091/seatunnel/actions/runs/39834451480) is now **completed with failure**, rather than queued/in progress. This updates the old guidance; I am not posting a second source review on the unchanged head `b659ea3f42116ab8003a10dc680fb784224ec49b`. The [inspected failed job](https://github.com/Nikk8091/seatunnel/actions/runs/39834451480/job/120023806314) reports `PreconditionViolationException: None of the supporting TestTemplateInvocationContextProviders ... provided a non-empty stream`. Specifically: HbaseIT (22 errors); representative all-connectors-it-2, all-connectors-it-6, and MySQL CDC logs have the same empty TestTemplate-provider signature. The transform-v2 part-2 lane also failed, but its log download timed out, so I am not claiming an inspected root cause for that lane. There is an upstream fix already in latest dev: [#12895](https://github.com/apache/seatunnel/pull/12895), commit `b65fe82b040c203ce669db83431916108f3e866e`. It adds `ContainerSelectionCondition` to skip a suite/method when every engine selected by `RUN_ENGINES` is explicitly disabled, instead of passing an empty invocation stream to JUnit. A genuinely empty container discovery still remains an error. I checked that this head does not contain that commit, and this PR does not change that shared E2E framework. The comparison is `diverged`, ahead 1 / behind 25 against dev, with no reported conflict. For this demonstrated upstream test-selection failure, please **sync latest dev and rerun the affected failed jobs first**. Do not copy unrelated framework changes into this PR or add dependencies to chase the old failure. Correctly skipped unsupported-engine cases are not runtime test passes; inspect the refreshed run for any other failures and preserve the relevant engine coverage. No outstanding source blocker from my previous full review. A successful current-head required Build and write-capable maintainer approval remain separate merge gates. Daniel has READ permission; this follow-up is not an effective approval. Thanks for keeping the patch 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]
