You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cxf.apache.org by "gnodet (via GitHub)" <gi...@apache.org> on 2026/03/12 16:45:13 UTC

[PR] Fix ExecutorService leak in WS-RM oneway tests [cxf]

gnodet opened a new pull request, #2963:
URL: https://github.com/apache/cxf/pull/2963

   ## Summary
   - `DeliveryAssuranceOnewayTest` and `MessageCallbackOnewayTest` create `ExecutorService` instances via `Executors.newSingleThreadExecutor()` in their `*AsyncExecutor` test methods but never shut them down
   - The leaked non-daemon threads prevent clean test completion, causing intermittent timeouts in CI (~95-100s for DeliveryAssurance, ~75s for MessageCallback)
   - Fix: track the `ExecutorService` in an instance field and shut it down in `tearDown()` with a 10-second grace period before forceful termination
   
   ## Test plan
   - [ ] CI passes without WS-RM oneway test timeouts
   - [ ] Both `DeliveryAssuranceOnewayTest` and `MessageCallbackOnewayTest` complete within their 60s timeout
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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: dev-unsubscribe@cxf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Fix ExecutorService leak in WS-RM oneway tests [cxf]

Posted by "coheigea (via GitHub)" <gi...@apache.org>.
coheigea merged PR #2963:
URL: https://github.com/apache/cxf/pull/2963


-- 
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: dev-unsubscribe@cxf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Fix ExecutorService leak in WS-RM oneway tests [cxf]

Posted by "coheigea (via GitHub)" <gi...@apache.org>.
coheigea commented on PR #2963:
URL: https://github.com/apache/cxf/pull/2963#issuecomment-4305390887

   I tried this locally and it doesn't seem to solve the flakiness of the tests


-- 
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: dev-unsubscribe@cxf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org