gnodet commented on PR #24337: URL: https://github.com/apache/camel/pull/24337#issuecomment-4863030147
_Claude Code on behalf of Guillaume Nodet_ Thanks for the review! Both points addressed: 1. **Heap histogram integrated**: Rebased on latest main and added `get_heap_histogram` to ToolRegistry. `camel_runtime_heap_histogram` in RuntimeTools now delegates to ToolRegistry like all other tools. (`camel_runtime_jfr_old_objects` doesn't appear to be on main yet — will integrate when it lands.) 2. **Topology defaults**: Replied on the inline comment — the change from `metric != null && metric` to `metric == null || metric` is intentional to align with ToolRegistry defaults (`true` when omitted). Added a behavioral note in the PR description. -- 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]
