ok2c commented on pull request #309:
URL: 
https://github.com/apache/httpcomponents-client/pull/309#issuecomment-868724887


   @rschmitt I came up with what I beleive is a better fix. I have also added 
test cases to verify correctness of new code.
   
   Essentially I was going about it all wrong. I was trying to ensure that 
request execution gets aborted as early as possible and by doing so I was 
creating more points of potential thread race conditions. Now I simplified 
things by letting the request execution procced to reach a point where it is 
easier to avoid race conditions and to ensure correct resource deallocation. 
Several chunks of ugly code is also now gone from `InternalExecRuntime`.
   
   Please review.


-- 
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]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to