CAMEL-23618: camel-tui - Add payload size metrics to Endpoints tab#23542
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Claus Ibsen <claus.ibsen@gmail.com>
Display body and header size columns (min/max/mean) in the Endpoints table when message size tracking is enabled. Add a mirrored sparkline chart showing IN vs OUT average body sizes over time. Add Reset Stats action to the F2 menu that resets route counters, endpoint registry stats, and all local sparkline history. Rename consumer TOTAL column to POLLS to clarify it tracks scheduler poll count, not exchange totals. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
gnodet
left a comment
There was a problem hiding this comment.
Adds payload size metrics (mean body size) tracking to the TUI Endpoints tab with sparkline history, plus a new "Reset Stats" action in the Actions menu. The reset logic properly clears both the server-side stats (via reset-stats action file) and all local sparkline history maps. RuntimeEndpointRegistry.reset() is also called in LocalCliConnector.doActionResetStatsTask().
Large diff (~800 lines), but mostly TUI rendering and history bookkeeping following the established pattern for other metric types (throughput, remote endpoints). Reviewed source code, config, and TUI integration — generated metadata skimmed.
LGTM.
Fully automatic review from Claude Code
…s tab Enable row navigation (up/down) in the Endpoints table with highlight. Add three-way chart mode toggle (a key): all, single, off. In single mode, the sparkline and flow diagram show rates for the selected endpoint URI (both IN and OUT directions). Per-endpoint rate history is tracked using compound keys (pid|uri) reusing the existing sliding-window delta logic. Also fix --sort=-size syntax in message-size example README. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
🧪 CI tested the following changed modules:
💡 Manual integration tests recommended:
All tested modules (14 modules)
|
Summary
Test plan
mvn installmessageSizeEnabled=true🤖 Generated with Claude Code