CryoThrust commented on PR #12156:
URL: https://github.com/apache/seatunnel/pull/12156#issuecomment-6416002538
Synced the existing branch with current `dev` in `7938f302f` using an
ordinary merge. The only conflict was in `StateTransitionCleanupTest`: both
sides' imports, cancellation tests and the upstream restore/metrics tests are
preserved. Relative to `dev`, this PR still changes only its original three
files; `PhysicalPlan` keeps the newer upstream stop/savepoint handling.
Validation on the merged head:
```sh
mvn -pl seatunnel-engine/seatunnel-engine-server -am \
-Dtest=JobStatusTest,StateTransitionCleanupTest \
-Dsurefire.failIfNoSpecifiedTests=false verify
```
All 40 reactor modules pass. `JobStatusTest`: 2 passed;
`StateTransitionCleanupTest`: 21 passed; no failures, errors or skips. Scoped
Spotless and the PR diff check pass. The command also completed the UI build;
integration tests retain the project's default skip and are not claimed here.
The new remote head is verified. Fork Actions still reports enabled, but
that setting alone does not establish that GitHub's usage-related workflow
pause is lifted. I have not counted absent checks as passes or created an empty
commit to trigger them. The separately tracked #12342 synchronization question
remains outside this change.
--
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]
