DanielLeens commented on PR #11613: URL: https://github.com/apache/seatunnel/pull/11613#issuecomment-5551384593
Quick CI-fact update on the current head (`1d815881620e`): my last comment noted the `Build` check was still `in_progress` and asked to confirm it finishes green before merging. It has since completed on the fork run - but not green: the run's overall conclusion is `cancelled` because one job, `engine-v2-it (11, ubuntu-latest)`, genuinely failed (1 test error inside the `SeaTunnel : E2E : Engine : Base` module), and a second job, `all-connectors-it-2 (11, ubuntu-latest)`, was cancelled as a result. The other 78 jobs all passed. This doesn't look related to this PR's REST-endpoint diff at first glance, and it isn't the usual apache-side pointer-dereference artifact I've flagged on other PRs - it's a real completed failure this time, not a pending/in-progress state. Worth triggering a fresh rerun of those two legs (or the whole `Build` workflow if easier) to get a clean signal before this comes out of Draft. No new code-side concern from me beyond what I already raised in my last full review. -- 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]
