Changes between Version 2 and Version 3 of Ticket #37177
- Timestamp:
- Jun 18, 2026, 7:51:56 AM (112 minutes ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #37177 – Description
v2 v3 44 44 and back. 45 45 46 We end up with essentially 16 context switches before reaching the view.46 We end up with essentially 16 context switches reaching the view and back. 47 47 48 48 If, instead, the default middleware are marked as `async_capable = False`, we get only 1 such transition during the middleware, as was the intent of the original feature. … … 85 85 I'm not sure we can just flip the flag — that's what I did for the benchmark — but maybe MiddlewareMixin could check to see if process_request/process_response were coroutine function or not before just declaring `True`? (There's probably a little more due diligence to do there too.) 86 86 87 I want to thank Mykhailo Havelia for pointing this issue out. We absolutely shouldn't be transitioning contexts multiple times each way in this case.87 I want to thank Mykhailo Havelia for pointing this issue out. The conclusion to remove the async support entirely goes too far I think, but we absolutely shouldn't be transitioning contexts multiple times each way in this case.