Skip to content

Throttle log messages#134

Merged
pglass merged 1 commit intomainfrom
pglass/throttled-logger
Aug 11, 2025
Merged

Throttle log messages#134
pglass merged 1 commit intomainfrom
pglass/throttled-logger

Conversation

@pglass
Copy link
Contributor

@pglass pglass commented Aug 11, 2025

What was changed

Throttle logs according to a single global max limit

  • Use Temporal's ThrottleLogger to throttle log messages
  • Add logging.throttleMaxRPS config option, which defaults to 10 rps

Why?

Avoid spamming millions of messages per day

Checklist

  1. Closes

  2. How was this tested:

  • Test with a branch that manually adds spammy logs in a few places: 34bef61
  1. Any docs updates needed?

@pglass pglass requested a review from a team as a code owner August 11, 2025 21:40
@pglass pglass requested review from hai719, meiliang86 and yux0 August 11, 2025 21:41
@pglass pglass merged commit 90cca0d into main Aug 11, 2025
6 checks passed
@pglass pglass deleted the pglass/throttled-logger branch August 11, 2025 21:56
hai719 pushed a commit to hai719/s2s-proxy that referenced this pull request Nov 24, 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