You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Actually I don't have any issue with the behavior that our ConnectionPool currently has. (let the connection expire on the server, limit number of connections and close above that limit)
I think we will likely want to later round that out with a timeout value too, but it doesn't seem strictly required, and I think we can close this off as "currently out of scope".
Activity
lovelydinosaur commentedon Jul 23, 2019
Actually I don't have any issue with the behavior that our ConnectionPool currently has. (let the connection expire on the server, limit number of connections and close above that limit)
I think we will likely want to later round that out with a timeout value too, but it doesn't seem strictly required, and I think we can close this off as "currently out of scope".
lovelydinosaur commentedon Dec 11, 2019
Actually for robustness, this really is required.