nielifeng commented on PR #11565: URL: https://github.com/apache/seatunnel/pull/11565#issuecomment-5088534186
Follow-up validation after review: - Expanded `isNoErrorCompleted()` coverage to the complete `CheckpointType x coordinator status x restored` matrix: 5 x 6 x 2 = 60 combinations, including missing state, non-final checkpoints, completed-point/savepoint, FINISHED/SUSPEND, and restored final checkpoints. - Fresh local result: full `CheckpointCoordinatorTest` 14/14 passed, failures/errors/skips 0/0/0. - `git diff --check` passed. - Independent code review found no P0-P3 issue and confirmed only the four intended combinations return true (final checkpoint + FINISHED/SUSPEND + not restored). The production change remains the minimal null-safe enum comparison; no additional production behavior was added. -- 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]
