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


   déjà vu, I've run into this flavor of leak a number of times because it's 
incredibly difficult to handle `cancel()` on any `Future<? extends 
AutoCloseable>`.
   
   > the core problem here is whose responsibility it is to release a pool 
entry back to the pool if the lease request simultaneously succeeds and gets 
cancelled.
   
   This exactly matches my experience, it's nontrivial to determine the 
singular 'owner' for a given future instance.


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

Reply via email to