gnodet commented on PR #26845: URL: https://github.com/apache/camel/pull/26845#issuecomment-5849295197
Thanks for the thorough review @davsclaus. All points addressed in bd4bc274615e: 1. **Null pool MXBean** — now logs at DEBUG when Hikari pool isn't started yet, instead of the misleading fallback message. 2. **`setAccessible(true)`** — dropped from production code; test stubs are now public static nested classes. 3. **Scope** — eviction restricted to the component's own `dataSource` only; removed `evictAllDataSourceConnections()` and the registry scan. No more cross-eviction or double-eviction when both jdbc and sql are in use. 4. **Credential refresh limitation** — documented in Javadoc and both component docs (jdbc/sql `.adoc`). The feature works only with pools that resolve credentials dynamically. 5. **FQCN** — `DefaultCamelContext` properly imported in both test files. -- 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]
