joshelser commented on pull request #149:
URL: https://github.com/apache/calcite-avatica/pull/149#issuecomment-874919148


   > In the (unlikely) case that a lot of pools are created, we are keeping 
those in memory even after they are unused.
   
   I was wondering about that, but couldn't come up with a plausible use-case 
that a customer may stumble onto such a problem. I agree that WeakReferences on 
the map values would be a nice improvement, but I don't think you need to lump 
that into this nice bug-fix.


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


Reply via email to