Skip to content

Kill process descendants on execution timeout - #56

Open
efegokdemir wants to merge 1 commit into
apache:mainfrom
efegokdemir:codex/issue-48-process-tree-timeout
Open

efegokdemir wants to merge 1 commit into
apache:mainfrom
efegokdemir:codex/issue-48-process-tree-timeout

Conversation

@efegokdemir

Copy link
Copy Markdown

Summary

  • Terminate descendants before the forked executor process when execution cleanup is required.
  • Keep Java 8 compatibility while using ProcessHandle on Java 9 and later.

Testing

  • mvn -pl maven-executor -am -DskipTests -Dspotless.check.skip=true -Dspotless.apply.skip=true compile — passed
  • git diff --check — passed

The module test run was started with the same Spotless skips but stopped after the existing ToolboxExecutorToolTest did not complete under the installed Java 27 runtime.

Fixes #48

Signed-off-by: Efe Gökdemir <efe@rexcode.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Forked executor leaves the process tree behind on timeout

1 participant