Skip to content

Conversation

@franz1981
Copy link
Contributor

This is optimizing http header creation

@franz1981 franz1981 marked this pull request as draft April 12, 2025 19:42
@franz1981 franz1981 marked this pull request as ready for review April 12, 2025 19:49
@franz1981
Copy link
Contributor Author

franz1981 commented Apr 12, 2025

This is using a timer and a scratch buffer/header for each connection opened.
It could be further improved (through a single encoded header repository) reducing by far the memory pressure and the number of required timer(s).

@franz1981
Copy link
Contributor Author

I think that the original code for this benchmark is broken because it doesn't stop the timer after a connection is closed. I will put this in draft and fix it first

@franz1981 franz1981 marked this pull request as draft April 12, 2025 21:10
@franz1981 franz1981 marked this pull request as ready for review April 13, 2025 16:26
@franz1981
Copy link
Contributor Author

This is ready to go as well since the problems were already present before and won't likely get any worse with this 🙏

@franz1981
Copy link
Contributor Author

@msmith-techempower ready to go 🙏

@msmith-techempower msmith-techempower merged commit fdcb171 into TechEmpower:master Apr 14, 2025
3 checks passed
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