Github user markt-asf commented on the issue:
https://github.com/apache/commons-pool/pull/4
Thanks for the PR. I've fixed this in a slightly different way after
reviewing the prior changes. The tst case I used largely as-is. Thanks.
---
--
Github user garydgregory commented on the issue:
https://github.com/apache/commons-pool/pull/4
I should be able to take a look tomorrow or the next day.
---
-
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.or
Github user cdeneux commented on the issue:
https://github.com/apache/commons-pool/pull/4
Hi @garydgregory,
To be able to release our product [Petals ESB](http://petals.ow2.org/), we
wish embed this fix. We can use the official released artefact, or our own
forked artefact.
Github user cdeneux commented on the issue:
https://github.com/apache/commons-pool/pull/4
Hi @garydgregory,
I have just updated the PR with a unit test validating the correct freeing
of eviction objects.
Regards,
Christophe
---
-
Github user garydgregory commented on the issue:
https://github.com/apache/commons-pool/pull/4
Hello @cdeneux !
Thank you for your PR.
Would you mind updating this PR with a unit test? It would be best if the
test failed without the changes to the main file.
Github user coveralls commented on the issue:
https://github.com/apache/commons-pool/pull/4
[![Coverage
Status](https://coveralls.io/builds/15805552/badge)](https://coveralls.io/builds/15805552)
Coverage increased (+0.06%) to 84.784% when pulling
**ec8b7769585bcb86a42a53
Github user coveralls commented on the issue:
https://github.com/apache/commons-pool/pull/4
[![Coverage
Status](https://coveralls.io/builds/15805552/badge)](https://coveralls.io/builds/15805552)
Coverage increased (+0.06%) to 84.784% when pulling
**ec8b7769585bcb86a42a53