ok2c commented on code in PR #728:
URL:
https://github.com/apache/httpcomponents-client/pull/728#discussion_r2596200761
##########
httpclient5/src/main/java/org/apache/hc/client5/http/impl/async/InternalHttpAsyncExecRuntime.java:
##########
@@ -66,6 +70,9 @@ static class ReUseData {
}
+ private static final ConcurrentMap<AsyncClientConnectionManager,
AtomicInteger> QUEUE_COUNTERS =
Review Comment:
@arturobernalg Do not see how this is any better. The static map is still
there and what you have is essentially a major resource leak right there.
--
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]
