wuxcer commented on issue #13358: URL: https://github.com/apache/trafficserver/issues/13358#issuecomment-4899804096
> Hi, I’ve opened a draft PR for the HTTP/2 re-enable path here: > > [#13362](https://github.com/apache/trafficserver/pull/13362) > > The change makes `Http2CommonSession::state_complete_frame_read()` schedule `HTTP2_SESSION_EVENT_REENABLE` on the session’s `NetVConnection` owner thread, rather than whichever thread happens to be holding the mutex at that point. > > I’ve kept this PR limited to the HTTP/2 path for now. I’d really appreciate any feedback on whether this looks like the right direction before I continue looking into the separate HTTP/1.1 path that runs after a cache write finishes. I've checked the PR code, and it makes sense. I would like to test this pr in my production environment. But it'll take a few days, you can keep going on with HTTP/1.1. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
