DanielLeens commented on PR #12904: URL: https://github.com/apache/seatunnel/pull/12904#issuecomment-6383286586
Thanks @wgzhao for the CI update and for answering @SEZ9's F1 to F8 questions with exact pointers. I confirmed the CI fact on the unchanged head `875a7ff788e9`: the fork run [wgzhao/seatunnel 39919297925](https://github.com/wgzhao/seatunnel/actions/runs/39919297925) has completed with conclusion `success` for this exact SHA, with no failed jobs. In my review of this head I found no source-level blocker and listed this Build as the only remaining gate from my side, so that gate is now met and my conclusion on `875a7ff788e9` is ready to merge. The two Low UI items in that review remain optional. The formal recheck you asked for in your last line is @SEZ9's own pass on their F1 to F8 threads; I am not closing those on their behalf. The final merge still needs an approval from a committer with write access. No further action is needed from you unless a new review comment comes in. -- 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]
