[GitHub] [trafficserver] SolidWallOfCode commented on pull request #4136: Restructure dispatchEvent and bring back most handlEvent calls

["SolidWallOfCode (GitHub)" <gi...@apache.org>]

Isn't that a blocking lock? That's not allowed in an `ET_NET` thread. If you're going to do a blocking lock at all, it would be better to do that on the original lock. Beyond that, this will call the handler of the continuation without a lock on the continuation's mutex, which is precisely what this PR is trying to prevent. Again, if it's OK to do that, why bother with any of this?

[ Full content available at: https://github.com/apache/trafficserver/pull/4136 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org