SEZ9 commented on PR #12165: URL: https://github.com/apache/seatunnel/pull/12165#issuecomment-6620851245
Thanks @Vivek1106-04 for posting the fork result for `b48e21019`. The `403 Forbidden` on the Maven wrapper download in `unit-test (17, windows-latest)` was runner infrastructure, not the change, so the attempt 2 `success` of run 41170040575 on the unchanged head `b48e2101972853a13914547d659e55b0e64f6ffb` is what I needed to see. On your question: please keep the two Low nits (the dead fallback scheduler and the bare `CheckpointManager` mock in `CheckpointCoordinatorTest`) in the agreed follow-up PR. No need to push them here; the reviewed scope of this PR stays as it is at `b48e21019`. Nothing further is needed from you on this one. A write-capable maintainer still needs to reconcile the required `Build` check and give the final approval for the merge gate. <!-- streview-comment:2257 --> -- 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]
