Remove c3p0 dependencies from installer code so we can change the DB connection pooling implementation
Description
c3p0 has some serious performance issues when working in very large environments. Would like to change to DB Pool but the installer has hooks directly into c3p0 vs. the DataSource Factory.
Acceptance / Success Criteria
None
Lucidchart Diagrams
Activity
Show:
Benjamin Reed March 7, 2011 at 11:00 AM
FYI, this got refactored a bit more. If you use an old configuration, it will still fall back to C3P0, but it is recommended that you add the new connection-pool tag:
Benjamin Reed March 1, 2011 at 5:27 PM
Finished in 1.9. No benchmarks, but it seems faster with the DBPool implementation compared to C3P0, even on my test machine with only a few nodes.
commit 5949f8cbdb1354bc3423b3f41c9aa046de40ec9a Author: Benjamin Reed <ranger@opennms.org> Date: Tue Mar 1 17:24:38 2011 -0500
- enable DBPool for connection pooling
Adds support for specifying the database connection factory in opennms-datasources.xml.
Add default opennms-datasources.xml config that points to C3P0ConnectionFactory as the default implementation.
Adds a connection factory implementation using DBPool.
c3p0 has some serious performance issues when working in very large environments. Would like to change to DB Pool but the installer has hooks directly into c3p0 vs. the DataSource Factory.