DanielLeens commented on PR #11398: URL: https://github.com/apache/seatunnel/pull/11398#issuecomment-4968431309
Rechecked the current head `5ac02727f834` against the latest `dev` after the CI / merge-gate fact changed. ## Runtime path rechecked ```text same runtime path as the previous Daniel review on this unchanged head ``` ## Merge conclusion ### Conclusion: still needs fixes before merge 1. Blocking items - The source-side blockers from my previous Daniel review still stand on this unchanged head. - Separately, this branch is behind upstream (`compare status: diverged`, `ahead_by=1`, `behind_by=10`), and the current failing gate is `FAILURE gate: Build`: https://github.com/apache/seatunnel/runs/86324931510 2. Lowest-cost next step - Please sync with the latest `dev` and rerun the failing gate first so we can separate baseline CI noise from the remaining code-side fixes. - After that, if the same source blocker is still unresolved, I would still keep this PR out of merge-ready state from Daniel's side. -- 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]
