Commit 884a70b
fix(blackhole sink): disable rate limiting for periodic stats messages (#24190)
* fix(blackhole sink): disable rate limiting for periodic stats messages
The blackhole sink's informational "Collected events" messages were being
rate-limited since rate limiting was enabled by default in #24024. This is
undesirable because:
1. These are deliberately scheduled periodic messages (controlled by the
`print_interval_secs` config option), not error conditions that could
flood the logs
2. Users explicitly configure the frequency - rate limiting defeats that
explicit configuration and breaks user expectations
3. The interval timer already prevents log flooding, making additional
rate limiting redundant
4. The blackhole sink is used for debugging/testing, where predictable
output is essential
This fix adds `internal_log_rate_limit = false` to both info! calls,
similar to how the console sink disables rate limiting for its critical
operational messages.
Fixes #24188
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <[email protected]>
* chore: add changelog fragment for blackhole sink rate limiting fix
---------
Co-authored-by: Claude <[email protected]>1 parent a011c1e commit 884a70b
File tree
2 files changed
+5
-0
lines changed- changelog.d
- src/sinks/blackhole
2 files changed
+5
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
65 | 65 | | |
66 | 66 | | |
67 | 67 | | |
| 68 | + | |
68 | 69 | | |
69 | 70 | | |
70 | 71 | | |
| |||
75 | 76 | | |
76 | 77 | | |
77 | 78 | | |
| 79 | + | |
78 | 80 | | |
79 | 81 | | |
80 | 82 | | |
| |||
0 commit comments