Skip to content

Show max time per request in statistics - #228

Merged
fcsonline merged 1 commit into
masterfrom
feature/show-max-stats
May 31, 2026
Merged

fcsonline merged 1 commit into
masterfrom
feature/show-max-stats

Conversation

@fcsonline

Copy link
Copy Markdown
Owner

Summary

Adds a Max time per request line to both the per-task and global statistics blocks. Previously the worst-case figure shown was the 99.9th percentile, which (as noted in #188) often is not enough to understand a system's tail behavior. The max is read straight from the existing hdrhistogram via max(), so there is no extra bookkeeping or memory cost.

Example output

ping                      99.9'th percentile        477ms
ping                      Max time per request      477ms
...
99.9'th percentile        477ms
Max time per request      477ms

Notes

  • cargo fmt --check, cargo clippy, and cargo build all pass.
  • Verified at runtime that the line renders in both the per-task and global blocks.

Closes #188

🤖 Generated with Claude Code

Add a "Max time per request" line to both the per-task and global
statistics blocks. The output previously stopped at the 99.9th
percentile, which hides the true worst-case latency. The value is
read from the existing hdrhistogram via `max()`, so no extra
bookkeeping is needed.

Closes #188

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@fcsonline
fcsonline merged commit ecb6d20 into master May 31, 2026
9 checks passed
@fcsonline
fcsonline deleted the feature/show-max-stats branch May 31, 2026 18:48
@fcsonline fcsonline mentioned this pull request May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show max in statistics

1 participant