CryoThrust commented on PR #12152: URL: https://github.com/apache/seatunnel/pull/12152#issuecomment-5994008997
Synced onto `dev` at `eb2eb1ddc` — the branch was `behind_by=22`, as you both noted. Rebase was clean (no conflicts in `CheckpointCoordinator.java` or its test), and I rebuilt the module dependencies since `seatunnel-common` had moved on. ``` CheckpointCoordinatorTest 19 run, 0 failures spotless:check clean ``` Head is now `0d31cf999`, so the fresh run should land on a head that includes everything merged to `dev` since the last push. On the two follow-ups you both flagged as non-blocking: agreed, tracking them separately alongside #12342 — the double-rejection last-resort branch (`CheckpointCoordinator.java:585-604`) is unexercised and unreachable today, and the exception message recorded on that path is worth a second look when someone does wire it up. -- 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]
