swaldman / c3p0

a mature, highly concurrent JDBC Connection pooling library, with support for caching and reuse of PreparedStatements.
http://www.mchange.com/projects/c3p0
Other
1.28k stars 338 forks source link

update unreturnedConnectionTimeout description in documentation #160

Closed matteomazza91 closed 6 months ago

matteomazza91 commented 2 years ago

This is an opinable proposal.

I'd like to not see this option being discouraged as I think it's necessary for increased resiliency in prod-ready applications. Still, the description should continue to point out that these kind of errors must be monitored and fixed.

swaldman commented 6 months ago

Thanks!

See https://github.com/swaldman/c3p0/commit/bce166efcbbd1e9a81c414cb5b722a1cc784998c for my editorializing on your editorializing.