Skip to content

fix(logs, metrics): Make ThreadedPeriodicWorker and TelemetryEventBuffer fork friendly - #3097

Merged
sl0thentr0py merged 2 commits into
masterfrom
neel/tel-buf-fork
Sep 28, 2026
Merged

sl0thentr0py merged 2 commits into
masterfrom
neel/tel-buf-fork

Conversation

@sl0thentr0py

@sl0thentr0py sl0thentr0py commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

No description provided.

@sl0thentr0py
sl0thentr0py changed the base branch from master to neel/tel-buf-close September 24, 2026 15:12
@sl0thentr0py sl0thentr0py changed the title neel/tel buf fork fix(logs, metrics): Make ThreadedPeriodicWorker and TelemetryEventBuffer fork friendly Sep 24, 2026
@sl0thentr0py
sl0thentr0py added this pull request to stack #3098 September 24, 2026 15:13

@dingsdax dingsdax left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does SessionFlusher also need a reset_if_forked override for pending_aggregates?

Base automatically changed from neel/tel-buf-close to master September 24, 2026 20:35
@sl0thentr0py

Copy link
Copy Markdown
Member Author

@dingsdax added

@sl0thentr0py
sl0thentr0py merged commit 91a075d into master Sep 28, 2026
158 checks passed
@sl0thentr0py
sl0thentr0py deleted the neel/tel-buf-fork branch September 28, 2026 11:45
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