DanielLeens commented on PR #12907: URL: https://github.com/apache/seatunnel/pull/12907#issuecomment-6607570367
Thanks @1328837476-hug for rerunning only the two failed jobs. I confirmed it on the unchanged head `02312129ac39c601593eeef80bce7db78f58a3a3`: the fork [Build](https://github.com/1328837476-hug/seatunnel/actions/runs/41461580862) is now `success` on attempt 2, the Apache-side `Build` check reports `pass`, and the PR signal workflow has passed. That confirms the `SeaTunnelClientTest.testGetMultiTableJobMetrics` assertion and the Doris container start-up timeout were unrelated to this diff. A green `Build` on this head was the only remaining gate in my last review, and no Ingres code changed, so I am approving this head. Real-server validation with a licensed Ingres instance remains the documented non-blocking follow-up, and a write-capable committer still needs to give the final approval. Thank you for the careful, well-scoped work! -- 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]
