ok2c commented on pull request #277: URL: https://github.com/apache/httpcomponents-core/pull/277#issuecomment-811407596
> Just out of curiosity, did you try synchronizing on the `leaseFuture` in `PoolingHttpClientConnectionManager`? @rschmitt This would only work with a prior knowledge of the future synchronizing on itself internally. If code duplication is the only concern, it could be dealt with extracting the common code into a shared class. I am more concerned that my original fix breaks the contract of the `Future` interface, but I see no way around it. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
