DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUGĀ· RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT <http://issues.apache.org/bugzilla/show_bug.cgi?id=39069>. ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED ANDĀ· INSERTED IN THE BUG DATABASE.
http://issues.apache.org/bugzilla/show_bug.cgi?id=39069 Summary: PoolGuardConnectionWrapper violates close() contract Product: Commons Version: unspecified Platform: All URL: http://svn.apache.org/repos/asf/jakarta/commons/proper/d bcp/trunk/src/java/org/apache/commons/dbcp/PoolingDataSo urce.java OS/Version: All Status: NEW Severity: normal Priority: P2 Component: Dbcp AssignedTo: commons-dev@jakarta.apache.org ReportedBy: [EMAIL PROTECTED] org.apache.commons.dbcp.PoolingDatasource.PoolGuardConnectionWrapper.close() violates the Connection.close() contract specified in the Java 1.5 API. The current API specifies that calling close() on an already-closed connection is a no-op. (Blame Sun for the bug. The API didn't used to say that.) PoolGuardConnectionWrapper.close() first calls checkOpen() which throws an exception if close() has already been called. Clearly that's not a no-op. The simplest fix is to change the first line in the close() method from this: checkOpen(); to this: if (this.delegate == null) return; As of today (2006-03-22) this bug is in the latest SVN source (and has been in previous versions as well). DelegatingConnection and PoolingConnection don't seem (from a quick glance) to have this problem. -- Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]