SEZ9 commented on PR #12568: URL: https://github.com/apache/seatunnel/pull/12568#issuecomment-5975887539
Thanks for the thorough investigation, @SEPURI-SAI-KRISHNA. The evidence for run `36966371906` on `314556b79` is clear: 82 successful, 0 failed, with the `MultiTableSinkWriterTest` additions green on all four unit-test legs, and the only non-success being `Build website` hitting the 90-minute `document` job timeout at the same point on all three attempts. I agree a fourth rerun would not change anything. That said, as I mentioned above, changes to the API core need a genuinely green check before merging, so let's go with your first option: 1. Wait for #12614 to land on `dev`. 2. Merge `dev` into this branch and push, so the new head picks up the fixed `backend.yml` (good catch that a rerun would reuse this branch's copy of the workflow). 3. Please keep the diff otherwise unchanged in that push, so the fresh run is a clean comparison against the current results. 4. Ping me once the run is green and I will take a final look and merge. No need for a separate CI issue or PR from you since that work is already in flight. Thanks for your patience on this one. <!-- streview-comment:1500 --> -- 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]
