DanielLeens commented on PR #12129: URL: https://github.com/apache/seatunnel/pull/12129#issuecomment-6338352321
Thanks @SEZ9. The reported passing `JobStatusTest` is useful evidence for the ordinal pinning change, and your remaining notes are explicitly non-blocking. Please keep their disposition in the PR description or a linked follow-up rather than introducing an unrelated fix just for this acknowledgement. One CI distinction still matters: a passing focused test in the separate verification run does not make the required fork Build green or attach it to this PR. The current Build check still requires a usable fork run for this head. Please link that run when available; my source-level conclusion remains unchanged, and the final repository approval/merge is for a write-capable maintainer. Status evidence: head `6d196d4c920792a0826bd4d783b7308875490d7a`, base `dev`, compare `diverged`, ahead 5, behind 10; GitHub mergeable state `blocked`. -- 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]
