shinrich opened a new pull request #7921: URL: https://github.com/apache/trafficserver/pull/7921
Fixes issue introduced in PR #7849. Found while running those changes against our 9.1.x on a production box this afternoon. Two times we triggered the assert in Http1ServerSesssion::release(). The state was set to KA_POOLED. The state was getting set to KA_POOLED both from the HttpSessionManager and within Http1ServerSession::release_transaction after calling httpSessionManager.release_session(). We are running with global pool. I think the session had been reassigned to another thread (the one that ultimately asserted) when the original thread set the state variable again. We should not be touching the server session again after releasing it in any case, and the change is unnecessary since it has already been set by the HttpSessionManager. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
