New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LDAP-316: support for apache commons-pool2 #351
Comments
Michael Osipov said: I am highly interested in this. Pool 1.x and 2.x are not compatible and different package names. |
Anindya Chatterjee said: Any update on this issue? Any road map for the implementation? If not yet, I'll work on it and create PR. |
Anindya Chatterjee said: I have submitted the PR as promised. [https://github.com//pull/33] |
Anindya Chatterjee said: Any update please about merging the PR? |
Rob Winch said: This is resolved in master |
@spring-issuemaster This issue still exists with latest Ldap core 2.3.2 release. When this will be fixed? |
I too have this issue right now with spring boot 2.2.3 |
@snytkine @salmanVD commons-pool2 is working fine for me with Spring LDAP Core version 2.3.2. One thing you have to consider is that there are two classes, one is for commons-pool and the other is for commons-pool2. For commons-pool 1.x use the class: For commons-pool 2.x use class: Note that the first one is in the pool subpackage while the second one on the pool2 subpackage. Also, the name of the commons-pool2 class starts with Pooled while the commons-pool 1 starts with Pooling The Spring LDAP documentation does not mention this difference in the Pooling Support section, I found out about this by looking at the code of spring-ldap-core. Sample code:
|
Cristi Vulpe (Migrated from LDAP-316) said:
I have tried to upgrade from commons-pool version 1.6 to commons-pool2 version 2.2 and I get this exception:
I have a bean that looks like this:
When I looked at the source code for PoolingContextSource.java I found that it is using the naming convention from commons-pool 1.x:
.
Is it possible to:
The text was updated successfully, but these errors were encountered: