-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
actix-http: play with allowing bytes passthrough in the h1 encoder #3368
base: master
Are you sure you want to change the base?
Conversation
I added a couple additional endpoints to the actix-web example in actix-http so I could run CPU: command: for note that this isn't testing streaming bodies |
probably semver-major now :) but this is mostly for comment on direction rather than an immediate request to merge |
related: BigBytes seems like a terrible name and I would love a better one :p |
PR Type
?????Bug?????? (reduces memory use)
PR Checklist
Overview
This is just me playing around with how to possibly avoid growing a persistent buffer to incredibly large sizes by allowing already-allocated Bytes types to pass through directly to IO in the h1 dispatcher
Comments very welcome
relates to #3367