DanielLeens commented on PR #12925: URL: https://github.com/apache/seatunnel/pull/12925#issuecomment-6397068846
Thanks for the update, @jjj-n, and for updating the PR description. I checked fork run 40072489701: attempt 2 is completed/success and bound to the unchanged head f3ca32eb. This matches my earlier review, which had no source-level blockers, so the CI gate I was waiting on is now met. The four Low items in my last review (Issues 1-4) remain optional; Issue 4 (per-test output isolation) is the most useful if you want a follow-up commit. The remaining gate is an approval from a committer with write access. Nothing was executed locally; this is based on code reading and the fork CI evidence. -- 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]
