You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "autumind (via GitHub)" <gi...@apache.org> on 2023/09/28 08:36:26 UTC
[GitHub] [commons-dbcp] autumind opened a new pull request, #309: Abort underlying connection of old driver may cause AbstractMethodError.
autumind opened a new pull request, #309:
URL: https://github.com/apache/commons-dbcp/pull/309
Use DelegatingConnection to abort underlying connection, in case of old driver may cause AbstractMethodError.
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Abort underlying connection of old driver may cause AbstractMethodError. [commons-dbcp]
Posted by "psteitz (via GitHub)" <gi...@apache.org>.
psteitz commented on PR #309:
URL: https://github.com/apache/commons-dbcp/pull/309#issuecomment-1979618958
Agreed
On Thu, Feb 29, 2024 at 11:55 AM Gary Gregory ***@***.***>
wrote:
> I can imagine that AbstractMethodError can happen for any JDBC API we call
> that is not implemented because the driver is too old. I think we can close
> this ticket. @psteitz <https://github.com/psteitz> ?
>
> —
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/commons-dbcp/pull/309#issuecomment-1971768741>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AALJJ2SWQAIKWXQAOE5UOB3YV54TNAVCNFSM6AAAAAA5KSWBYCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZRG43DQNZUGE>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Abort underlying connection of old driver may cause AbstractMethodError. [commons-dbcp]
Posted by "autumind (via GitHub)" <gi...@apache.org>.
autumind commented on PR #309:
URL: https://github.com/apache/commons-dbcp/pull/309#issuecomment-1751645431
> Needs a test or it will be a regression waiting to happen. Do you have a stack trave? Which driver? Which version? Which OS? Java version? There is only so much bending over backward we should do IMO. Can this be solved by updating the driver to a more recent version?
The driver is
> Needs a test or it will be a regression waiting to happen. Do you have a stack trave? Which driver? Which version? Which OS? Java version? There is only so much bending over backward we should do IMO. Can this be solved by updating the driver to a more recent version?
The driver is `com.springsource.oracle.jdbc.jar`
MANIFEST.MF
```
Manifest-Version: 1.0
Ant-Version: Apache Ant 1.6.5
Created-By: 1.5.0_51-b10 (Sun Microsystems Inc.)
Implementation-Vendor: Oracle Corporation
Implementation-Title: JDBC
Implementation-Version: 11.2.0.4.0
Repository-Id: JAVAVM_11.2.0.4.0_LINUX.X64_RELEASE
Specification-Vendor: Sun Microsystems Inc.
Specification-Title: JDBC
Specification-Version: 4.0
Main-Class: oracle.jdbc.OracleDriver
sealed: true
Name: oracle/sql/converter/
Sealed: false
Name: oracle/sql/
Sealed: false
Name: oracle/sql/converter_xcharset/
Sealed: false
Name: oracle/replay/driver/
Sealed: false
```
As you said, updating the driver can solve the issue. However, I've noticed `PoolableConnectionFactory.java` destroy `ABANDONED PoolableConnection` with underlying connection, what's the purpose of this operation.
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [commons-dbcp] garydgregory commented on pull request #309: Abort underlying connection of old driver may cause AbstractMethodError.
Posted by "garydgregory (via GitHub)" <gi...@apache.org>.
garydgregory commented on PR #309:
URL: https://github.com/apache/commons-dbcp/pull/309#issuecomment-1738987884
Needs a test or it will be a regression waiting to happen. Do you have a stack trave? Which driver? Which version? Which OS? Java version? There is only so much bending over backward we should do IMO. Can this be solved by updating the driver to a more recent version?
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Abort underlying connection of old driver may cause AbstractMethodError. [commons-dbcp]
Posted by "garydgregory (via GitHub)" <gi...@apache.org>.
garydgregory closed pull request #309: Abort underlying connection of old driver may cause AbstractMethodError.
URL: https://github.com/apache/commons-dbcp/pull/309
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Abort underlying connection of old driver may cause AbstractMethodError. [commons-dbcp]
Posted by "garydgregory (via GitHub)" <gi...@apache.org>.
garydgregory commented on PR #309:
URL: https://github.com/apache/commons-dbcp/pull/309#issuecomment-1979623707
Closing.
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Abort underlying connection of old driver may cause AbstractMethodError. [commons-dbcp]
Posted by "garydgregory (via GitHub)" <gi...@apache.org>.
garydgregory commented on PR #309:
URL: https://github.com/apache/commons-dbcp/pull/309#issuecomment-1971768741
I can imagine that AbstractMethodError can happen for any JDBC API we call that is not implemented because the driver is too old. I think we can close this ticket. @psteitz ?
--
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: issues-unsubscribe@commons.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org