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

feat(gtest): Use AuxiliaryTaskPool for current scheduled ops on gtest #4154

Merged
merged 9 commits into from
Aug 14, 2024

Conversation

techraed
Copy link
Member

Successor of the #4141, that actually uses the AuxiliaryTaskPool for the currently available scheduled tasks on the gtest, which are: 1) messages wait-wake, 2) delayed sending.

Note: the implementation of the delayed sending and the messages wait-wake proc is the same, but scheduled to be made more accurate.

Done:

  • actually use aux task pool in ExtManager::process_tasks
  • use task pool for storing the scheduled tasks for delayed sending and waking messages
  • move JournalHandler and TaskHandler traits impls to corresponding modules.

@techraed techraed added A0-pleasereview PR is ready to be reviewed by the team D5-tooling Helper tools and utilities labels Aug 14, 2024
@techraed techraed self-assigned this Aug 14, 2024
@techraed
Copy link
Member Author

@LouiseMedova this fixes the bug you found for the unreserved system reservation on the message to be consumed

@techraed techraed merged commit 5b6bb0d into master Aug 14, 2024
19 checks passed
@techraed techraed deleted the st-introduce-gtest-task-pool branch August 14, 2024 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A0-pleasereview PR is ready to be reviewed by the team D5-tooling Helper tools and utilities
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants