davsclaus commented on PR #24904: URL: https://github.com/apache/camel/pull/24904#issuecomment-5015688025
_Claude Code on behalf of davsclaus_ Thanks for the review! Addressed observation #2 — lifecycle strategy is now explicitly removed in `doStop()` and the field is reset to `null`, so it gets properly re-registered on restart. See d84f42edc76c. On the other observations: - **Thread safety**: Agreed it's fine in practice since route lifecycle transitions are sequential. - **Assertion style**: The existing LRA tests all use JUnit `Assertions` (zero `assertThat` usage across the module), so kept consistent with the existing pattern. -- 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]
