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.

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

Rough implementation of task poll callbacks #7107

Closed
wants to merge 12 commits into from

Conversation

rcoh
Copy link
Contributor

@rcoh rcoh commented Jan 17, 2025

_Just throwing this up for initial discussion, I need to do work to clean up internal names & get all the right tokio_unstable gating.

  • Implement on current thread worker
  • Implement on alt runtime
  • fix unstable
  • more tests and docs

Motivation

We would like to be able to store metadata before polls start and stop to allow us to instrument all polls, not just polls where you have spawned the task.

Solution

Add callbacks for poll start and stop

@github-actions github-actions bot added R-loom-current-thread Run loom current-thread tests on this PR R-loom-multi-thread Run loom multi-thread tests on this PR R-loom-multi-thread-alt Run loom multi-thread alt tests on this PR labels Jan 17, 2025
@rcoh rcoh closed this Jan 17, 2025
@rcoh rcoh reopened this Jan 17, 2025
@rcoh rcoh marked this pull request as draft January 17, 2025 20:41
@rcoh rcoh force-pushed the task-poll-callbacks branch 2 times, most recently from 4cb979a to e7c632a Compare January 17, 2025 21:05
@GJason88 GJason88 mentioned this pull request Jan 22, 2025
@rcoh rcoh force-pushed the task-poll-callbacks branch from e7c632a to 87c572d Compare January 24, 2025 14:53
@rcoh
Copy link
Contributor Author

rcoh commented Jan 24, 2025

closing in favor of #7120

@rcoh rcoh closed this Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R-loom-current-thread Run loom current-thread tests on this PR R-loom-multi-thread Run loom multi-thread tests on this PR R-loom-multi-thread-alt Run loom multi-thread alt tests on this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant