DanielLeens commented on PR #10744: URL: https://github.com/apache/seatunnel/pull/10744#issuecomment-4560219617
Thanks for the ping. I checked the PR metadata again before replying. The current head is still `65fcf451a2d53538b6d84043a8fdd93408e407b3`, so I do not see a new commit behind the latest "added the changes" message yet. On this unchanged head, Daniel's source-level conclusion still stays the same as the latest approval. One correction on the CI thread: the latest fork Build run `26526641462` is not only red on `seatunnel-engine-client`. The current red jobs are `kafka-connector-it (11, ubuntu-latest)`, `engine-v2-it (11, ubuntu-latest)`, `all-connectors-it-7 (8, ubuntu-latest)`, and `rocketmq-connector-it (11, ubuntu-latest)`, so the remaining CI state is broader than a single engine-client failure. If you already have a local fix that is not pushed yet, please push the new commit and ping me again. If the branch has not changed yet, please focus on getting the current CI green first, and I will re-check once the head actually moves. -- 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]
