Skip to content

Stats for discard-timeout and wait-limit - #1159

Merged
gthess merged 5 commits into
masterfrom
stat-discard-timeout
Nov 1, 2024
Merged

Stats for discard-timeout and wait-limit#1159
gthess merged 5 commits into
masterfrom
stat-discard-timeout

Conversation

@wcawijngaards

Copy link
Copy Markdown
Member

The new discard-timeout and wait-limit options have been accounted as requestlist_exceeded in the statistics. But this mixes them with the request exceed number. The change makes their own statistics counters for them. The new statistics counters are num.queries_discard_timeout and num.queries_wait_limit.

@wcawijngaards wcawijngaards self-assigned this Oct 23, 2024
@wcawijngaards
wcawijngaards requested a review from gthess October 23, 2024 09:42

@gthess gthess left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, useful statistics to differentiate between performance drops and configuration drops of packets.

@gthess
gthess merged commit 75967e7 into master Nov 1, 2024
@gthess
gthess deleted the stat-discard-timeout branch November 1, 2024 14:53
gthess added a commit that referenced this pull request Nov 1, 2024
- Some clean up for stat_values.test.
@edmonds edmonds mentioned this pull request Nov 13, 2025
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.

2 participants