li3zhi4 commented on PR #11677: URL: https://github.com/apache/seatunnel/pull/11677#issuecomment-6123218841
@DanielLeens @SEZ9 — the remaining Low item in review 5479163539 is now addressed in `bedcc58b4a`: the recovery comment no longer claims that observing RUNNING means replay has completed. It states that the status wait only confirms the job is alive, and the final source-to-sink equality assertion is the convergence guard. This is a comment-only change; no assertion or runtime behavior was weakened. `git diff --check` passed; I did not present this as a new live test run. One pointer correction to my previous reply: the injected-failure **log wait** is at `AbstractMysqlCDCITBase.java:265-279`, and it uses the two-minute budget. Line 227 is the separate completed-checkpoint wait. The log-length clamp is still at line 275. The trigger is removed in `finally` at lines 280-282; retry=3 deliberately tolerates a replay arriving before removal, rather than claiming atomic ordering. The current head retains the validated production fix and all F1-F8 changes. Remote Build dispatch and a current write-enabled approval remain separate gates; they are not satisfied by this comment-only update. -- 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]
