Hi,

Following https://issues.apache.org/jira/browse/OFBIZ-2599 and 
https://issues.apache.org/jira/browse/DBCP-294
I have build both
the last DBCP available revision (commons-dbcp.jar)
and
commons-dbcp-1.3-20091111-r829616.jar

Should we continue to follow the previous way or should we not include the lastest ? As some fixes have been added between I'd prefer the lastest.
There have been the following changes between

________________________________________________________________________________________
Revision: 834702
Author: markt
Date: 00:01:56, mercredi 11 novembre 2009
Message:
Fix DBCP-303. Patch provided by Dave Oxley.
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/cpdsadapter/PooledConnectionImpl.java
Modified : /commons/proper/dbcp/trunk/xdocs/changes.xml

Revision: 834346
Author: psteitz
Date: 06:05:54, mardi 10 novembre 2009
Message:
Made inner class static.
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/jocl/JOCLContentHandler.java

Revision: 834335
Author: psteitz
Date: 05:19:12, mardi 10 novembre 2009
Message:
Added test case verifying that prepared statement pool acts as LRU cache.
----
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestPStmtPoolingBasicDataSource.java

Revision: 832634
Author: psteitz
Date: 01:31:50, mercredi 4 novembre 2009
Message:
Made PoolGuardConnectionWrapper static, eliminated possible NPE in toString.
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java

Revision: 831723
Author: psteitz
Date: 19:35:35, dimanche 1 novembre 2009
Message:
Added findbugs exclude filter.
----
Added : /commons/proper/dbcp/trunk/findbugs-exclude-filter.xml
Modified : /commons/proper/dbcp/trunk/pom.xml

Revision: 831707
Author: psteitz
Date: 18:02:29, dimanche 1 novembre 2009
Message:
Avoid possible NPE on toString().
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDataSource.java

Revision: 831703
Author: psteitz
Date: 17:52:19, dimanche 1 novembre 2009
Message:
Made prepared statement key inner class static.
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingConnection.java

Revision: 831690
Author: psteitz
Date: 16:51:29, dimanche 1 novembre 2009
Message:
Added null statement key field check in makeObject.
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingConnection.java

Revision: 831576
Author: psteitz
Date: 19:13:14, samedi 31 octobre 2009
Message:
Removed incorrectly advertised exception. JIRA: DBCP-302.  Thanks to Sebastian 
Bazley.
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/jocl/JOCLContentHandler.java
Modified : /commons/proper/dbcp/trunk/xdocs/changes.xml

Revision: 830174
Author: markt
Date: 14:30:50, mardi 27 octobre 2009
Message:
Add a test case for DBCP-265
----
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestAll.java
Added : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestDelegatingDatabaseMetaData.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TesterConnection.java
Added : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TesterDatabaseMetaData.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TesterResultSet.java

Revision: 830170
Author: markt
Date: 14:28:48, mardi 27 octobre 2009
Message:
Complete the fix for DBCP-265
I noticed this gap when writing the Javadoc for DelegatingDatabaseMetaData
Test case to follow
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/DelegatingConnection.java

Revision: 830162
Author: markt
Date: 14:18:54, mardi 27 octobre 2009
Message:
Add some JavaDoc
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/DelegatingDatabaseMetaData.java

Revision: 830136
Author: sebb
Date: 12:28:05, mardi 27 octobre 2009
Message:
Null checks
----
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestConnectionPool.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestPoolableConnection.java

Revision: 830135
Author: sebb
Date: 12:25:13, mardi 27 octobre 2009
Message:
assertTrue(var != null) => assertNotNull(var)
- simplifies code
- avoids later warnings of possible NPE
----
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestBasicDataSource.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestConnectionPool.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestPoolableConnection.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/cpdsadapter/TestDriverAdapterCPDS.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/datasources/TestPerUserPoolDataSource.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/datasources/TestSharedPoolDataSource.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/managed/TestManagedDataSourceInTx.java

Revision: 830130
Author: sebb
Date: 12:03:06, mardi 27 octobre 2009
Message:
Add AL header
----
Modified : /commons/proper/dbcp/trunk/doap_dbcp.rdf

Revision: 830126
Author: markt
Date: 11:45:48, mardi 27 octobre 2009
Message:
Fix line endings and add missing svn properties
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/managed/PoolableManagedConnection.java
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/managed/PoolableManagedConnectionFactory.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/cpdsadapter/TestDriverAdapterCPDS.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/datasources/TestCPDSConnectionFactory.java

Revision: 830125
Author: markt
Date: 11:36:25, mardi 27 octobre 2009
Message:
Fix various Checkstyle/Eclipse warnings in the test cases including
- remove unused imports
- remove unused local variables
- mark methods as deprecated if they override deprecated methods
- avoid auto-boxing

All unit tess still pass
----
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestBasicDataSource.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestDriverManagerConnectionFactory.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestManual.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestPStmtPooling.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TesterCallableStatement.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/cpdsadapter/TestDriverAdapterCPDS.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/datasources/PooledConnectionProxy.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/datasources/TestCPDSConnectionFactory.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/datasources/TestInstanceKeyDataSource.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/datasources/TestPerUserPoolDataSource.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/managed/TestBasicManagedDataSource.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/jocl/TestAll.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/jocl/TestJOCLContentHandler.java

Revision: 830110
Author: markt
Date: 10:55:10, mardi 27 octobre 2009
Message:
Remove unused import
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/datasources/KeyedCPDSConnectionFactory.java

Revision: 829661
Author: markt
Date: 22:41:41, dimanche 25 octobre 2009
Message:
Add recent fixes.
----
Modified : /commons/proper/dbcp/trunk/xdocs/changes.xml

Revision: 829659
Author: markt
Date: 22:37:10, dimanche 25 octobre 2009
Message:
Fix DBCP-203. Provide an option to specify the class loader to be used to load 
the JDBC driver for a BasicDataSource.
Based on a patch suggested by Mark Grand.
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/BasicDataSource.java

Revision: 829653
Author: markt
Date: 22:16:18, dimanche 25 octobre 2009
Message:
Extend fix for DBCP-8 to PerUserPoolDataSource
The fix is a little ugly to maintain binary compatibility in PoolKey - I really 
wanted to add password as a separate field
Unit tests all pass with fix applied
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/datasources/PerUserPoolDataSource.java
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/datasources/PoolKey.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/datasources/TestPerUserPoolDataSource.java

Revision: 829642
Author: psteitz
Date: 21:36:14, dimanche 25 octobre 2009
Message:
Improved thread-safety.
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/managed/BasicManagedDataSource.java
Modified : 
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/managed/TestBasicManagedDataSource.java

Revision: 829623
Author: psteitz
Date: 19:19:37, dimanche 25 octobre 2009
Message:
Made XADataSource configurable in BasicManagedDataSource.
JIRA: DBCP-289
Reported and patched by Marc Kannegießer.
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/managed/BasicManagedDataSource.java
Modified : /commons/proper/dbcp/trunk/xdocs/changes.xml

Revision: 829618
Author: psteitz
Date: 17:57:18, dimanche 25 octobre 2009
Message:
Removed deprecated exception class prior to first release of 
BasicManagedDataSourc.
----
Modified : 
/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/managed/BasicManagedDataSource.java


Thanks and sorry for the long post

Jacques

Reply via email to