Skip to content

Use total requests for throughput calculation#530

Merged
sjmonson merged 1 commit intomainfrom
fix/console_total_throughput
Jan 16, 2026
Merged

Use total requests for throughput calculation#530
sjmonson merged 1 commit intomainfrom
fix/console_total_throughput

Conversation

@sjmonson
Copy link
Collaborator

Summary

Switches "Server Throughput Statistics" table in console output to use total request pool rather then just successful. See #529 for reasoning.

Related Issues


  • "I certify that all code in this PR is my own, except as noted below."

Use of AI

  • Includes AI-assisted code completion
  • Includes code generated by an AI application
  • Includes AI-generated tests (NOTE: AI written tests should have a docstring that includes ## WRITTEN BY AI ##)

@sjmonson sjmonson added this to the v0.5.2 milestone Jan 15, 2026
Signed-off-by: Samuel Monson <smonson@redhat.com>
@sjmonson sjmonson force-pushed the fix/console_total_throughput branch from 184df17 to aaacf4b Compare January 16, 2026 15:49
Copy link
Collaborator

@jaredoconnell jaredoconnell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only reason this would not be okay is if it was misleading, but the title doesn't seem misleading, so I'm approving.

@sjmonson sjmonson merged commit 8a81f13 into main Jan 16, 2026
29 of 30 checks passed
@sjmonson sjmonson deleted the fix/console_total_throughput branch January 16, 2026 17:10
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.

Console reported token throughput should use total requests

2 participants