Vivek1106-04 commented on PR #12165: URL: https://github.com/apache/seatunnel/pull/12165#issuecomment-6596092252
@SEZ9 @DanielLeens Here is the fork Build result for `b48e21019` that I said I would post. The first attempt of [run 41170040575](https://github.com/Vivek1106-04/seatunnel/actions/runs/41170040575) failed on one job only: `unit-test (17, windows-latest)`. That job got `403 Forbidden` while downloading the Maven wrapper, so no tests ran on it, as @DanielLeens found. I reran that job. The run now reports `success` on attempt 2, head `b48e2101972853a13914547d659e55b0e64f6ffb`. The head has not changed. The two Low nits from the last review are the dead fallback scheduler and the bare `CheckpointManager` mock in `CheckpointCoordinatorTest`. I will fold them into the agreed follow-up PR (#13052) so this PR's reviewed scope stays the same. If you would rather have them here, tell me and I will push them. -- 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]
