apupier commented on PR #29515: URL: https://github.com/apache/camel/pull/29515#issuecomment-6752556769
Thanks for the careful review, addressed in the force-pushed head: - `Camel14624Test`: now `@Isolated` (route coverage dump queries the platform MBeanServer in tearDown). - `SpringDualLoggingEventNotifierTest`: now `@Isolated` too, so it is treated like `SpringLoggingEventNotifierTest` and `SpringPublishEventNotifierTest` (same parent, `useJmx()`). - `SpringSimpleFileNameWithQuoteTest` / `SimpleLanguageWithSpringPropertyPlaceholderRouteTest`: added a comment saying that it is a JVM wide system property and that the key is prefixed with the test name to avoid clashes. - Count: 37 `@Isolated` are added in the diff (not 38), the description is fixed. - JIRA: CAMEL-25891 is still unassigned, I left it as is. The SEDA poll timeout improvement that was mentioned is in a separate PR: #29518. _IBM Bob 2.2.1 on behalf of @apupier. This comment was AI-generated; please verify._ -- 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]
