suggestion: tail latency with goSched by AlliBalliBaba · Pull Request #2033 · php/frankenphp
Navigation Menu
{{ message }}
php / frankenphp Public
- Notifications You must be signed in to change notification settings
- Fork 444
Merged
Conversation
Copy link Copy Markdown
Contributor
AlliBalliBaba
commented
Nov 25, 2025
AlliBalliBaba
commented
Alternate implementation to #2016 that doesn't reduce RPS with lower amounts of threads
AlliBalliBaba added 5 commits
November 24, 2025 22:27
AlliBalliBaba
mentioned this pull request
Closed
withinboredom approved these changes Nov 25, 2025
Copy link Copy Markdown
Member
withinboredom
left a comment
withinboredom
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not just put it at the top of servHttp()?
henderkes approved these changes Nov 25, 2025
Copy link Copy Markdown
Contributor Author
The request might also get rejected, feels most appropriate right before the select that it's supposed to make more fair.
dunglas approved these changes Nov 26, 2025
dunglas
deleted the
suggestion/dispatch-latency
branch
Copy link Copy Markdown
Member
dunglas
commented
Nov 26, 2025
dunglas commented
Nov 26, 2025Thank you, everyone, for figuring this out! Impressive teamwork.
Th3Mouk
mentioned this pull request
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants