DanielLeens commented on PR #10799: URL: https://github.com/apache/seatunnel/pull/10799#issuecomment-4707067979
Thanks for the update, and thanks @nzw921rx for confirming that the remaining red signal is the known flaky `SeaTunnelEngineClusterRoleTest.testWorkerIsFirstMemberThenGetJobDetailStatus` rather than the CDC path touched by this PR. Since the head is still `d79f2a5f895a` and there is no new code delta after Daniel's last source-level pass, I'm keeping this as a reply-only follow-up rather than starting another full rereview on the same revision. From Daniel's side, I do not have a new source-level blocker on the current head. The practical next step looks like: rebase onto the latest `dev` as suggested, rerun only the failed task(s), and then let maintainers make the merge call once that flaky gate is green again. -- 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]
