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
proposal: net/http: add Request.SetContext #48811
Comments
Copying a comment from CL 349369:
|
For the std, the prohibition for But for non-std frameworks that want to be compatible with std Handler, this limits the implementation of non-std frameworks. Although I've found a way here, it wastes more. The std solution——one or more goroutines per connection, when handling a 1000k scenario, cost huge memory, scheduling, GC, and leads to STW more easily and it even performs obviously worse than |
I'm sorry, but I don't see what |
Sorry for that, I forgot to explain. Here I want to set the |
nbio is a poller framework, it saves goroutines and aims to solve 1000k, also wants to be compatible with std Handler as much as possible, so I want to set the |
Request is explicitly immutable. That's why we have Request.WithContext. |
Declining as infeasible (contradicts basic design). |
This proposal has been declined as infeasible. |
related: #48339
The text was updated successfully, but these errors were encountered: