DanielLeens commented on PR #10799: URL: https://github.com/apache/seatunnel/pull/10799#issuecomment-4446600179
Hi @vsantonastaso, I rechecked the current PR state again before replying. The head is still `78799b058ced0bd449fce56458443bbcc53bc662`, so there is still no new commit after Daniel approved this exact version. Because of that, I am not starting another full source re-review on the same head. From Daniel's side, I still do not see a reopened code-path blocker on the current source revision. The remaining visible gate is CI: the latest `Build` on this same head is still red, and the fork run linked from the check is failing across a wide integration spread, including multiple `all-connectors-it-*` jobs, `jdbc-connectors-it-ddl`, `oracle-cdc-connector-it`, `doris-connector-it`, and `paimon-connector-it`, with several follow-on jobs cancelled after those failures. So my current position is unchanged: I would not call the PR merge-ready until the latest `Build` is green, or until the failing jobs are narrowed down to a clearly unrelated infrastructure signal on a fresh rerun. If you push a new commit or get a cleaner CI signal, please ping me again and I will recheck the newest state carefully. -- 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]
