DanielLeens commented on PR #12909: URL: https://github.com/apache/seatunnel/pull/12909#issuecomment-6397053338
Thanks for your patience. The CI fact in my previous review has changed: the required [Build](https://github.com/apache/seatunnel/runs/120910010816) is now SUCCESS for unchanged head `51c3cfabc9e3c8cfe6f8925481a7f6d352c891f5`. The earlier pending-Build statement is superseded; I still have no source-side blocker from my previous complete review. The remaining merge gate is **the actual merge conflict**: GitHub currently reports DIRTY against `dev`. Please sync with the latest `dev` and resolve the conflicts, preserving the entries already on the base. This request is for a confirmed content conflict, not a generic behind-branch CI suggestion. A changed head will need its own fresh checks; the green result above belongs only to the stated head. Compare metadata is `diverged`, ahead 2 / behind 8. Final write-capable maintainer approval is also required. Because the conflict remains, this is not a CI-only situation in which green checks remove every gate; I am not claiming APPROVED or asking you to change the already-reviewed source logic. This is a gate-status correction, not another full source review. -- 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]
