davsclaus opened a new pull request, #24834:
URL: https://github.com/apache/camel/pull/24834
## Summary
- `SagaProcessor.doStop()` was calling
`ServiceHelper.stopService(sagaService)` on the context-level
`CamelSagaService`. Since this service is a singleton shared by all saga
routes, stopping one route would shut down the saga infrastructure (executor,
LRA client) for the entire CamelContext — silently breaking compensation on all
other routes.
- Removed both `doStart()` and `doStop()` overrides from `SagaProcessor`.
The saga service lifecycle is managed by the CamelContext that owns it, not by
individual route processors.
- Added `SagaSharedServiceRouteStopTest` verifying that stopping one saga
route keeps the shared service running and compensation on other routes still
works.
## Test plan
- [x] New `SagaSharedServiceRouteStopTest` with two tests:
- `testStoppingOneSagaRouteKeepsSharedServiceRunning` — verifies the
shared `InMemorySagaService` stays started after stopping a single saga route
- `testCompensationOnOtherRouteStillWorksAfterStoppingOneSagaRoute` —
verifies compensation endpoint is invoked on route-b after route-a is stopped
- [x] All existing saga tests pass (`SagaTest`, `SagaPropagationTest`,
`SagaOptionsTest`, `SagaFailuresTest`, `SagaRemoveHeadersTest`,
`SagaTimeoutTest`)
_Claude Code on behalf of davsclaus_
🤖 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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
