Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

add tldrs backend to TEF #31

Merged
merged 6 commits into from
Aug 23, 2024
Merged

add tldrs backend to TEF #31

merged 6 commits into from
Aug 23, 2024

Conversation

c-cube
Copy link
Owner

@c-cube c-cube commented Aug 23, 2024

  • break tef: use mtime.now, not a counter, for multiproc
  • feat tef: accept a out_channel directly in the collector
  • feat: add trace-tef.tldr for tracing multiple processes
  • fix trace-tef.tldr: turn TEF path to an absolute one
  • add basic bench for trace-tef.multiproc
  • rename tef.tldr to tef.tldrs

wintersteiger
wintersteiger approved these changes Aug 23, 2024
View reviewed changes
Copy link

@wintersteiger wintersteiger left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM, especially with all the changes we made earlier today to the other side of it!

@c-cube c-cube merged commit 57aec09 into main Aug 23, 2024
0 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wintersteiger wintersteiger wintersteiger approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@c-cube @wintersteiger