yes tomcat-jdbc is another pool impl than dbcp2. tomcat shades dbcp2 at build time but also provides jdbc impl.
Romain Manni-Bucau @rmannibucau <https://twitter.com/rmannibucau> | Blog <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> | LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber <http://www.tomitribe.com> 2015-08-03 17:00 GMT+02:00 Jean-Louis Monteiro <jlmonte...@tomitribe.com>: > Hard to say for may. > My understanding is that DBCP 2 and Tomcat JDBC finally merged together and > Tomcat is now using DBCP2. > > Am I wrong? > > -- > Jean-Louis Monteiro > http://twitter.com/jlouismonteiro > http://www.tomitribe.com > > On Mon, Aug 3, 2015 at 7:58 AM, Romain Manni-Bucau <rmannibu...@gmail.com> > wrote: > > > For tracking: https://issues.apache.org/jira/browse/TOMEE-1620 > > > > migrating to tomcat ones would be as easy as a sed then > > > > > > Romain Manni-Bucau > > @rmannibucau <https://twitter.com/rmannibucau> | Blog > > <http://rmannibucau.wordpress.com> | Github < > > https://github.com/rmannibucau> | > > LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber > > <http://www.tomitribe.com> > > > > 2015-08-02 20:12 GMT+02:00 Romain Manni-Bucau <rmannibu...@gmail.com>: > > > > > Now build is green next question : do we replace dbcp by version 2? I > am > > > almost tempted to not use dbcp but the shade of tomcat even in openejb > > just > > > to avoid the double dep in tomcat. Wdyt? > > > Le 31 juil. 2015 17:11, "Romain Manni-Bucau" <rmannibu...@gmail.com> a > > > écrit : > > > > > >> pushed a first version linked to this jira: > > >> https://issues.apache.org/jira/browse/TOMEE-1617 > > >> > > >> Only connection has defaults but no more the pooling itself (size, > > >> eviction etc). > > >> > > >> > > >> Romain Manni-Bucau > > >> @rmannibucau <https://twitter.com/rmannibucau> | Blog > > >> <http://rmannibucau.wordpress.com> | Github > > >> <https://github.com/rmannibucau> | LinkedIn > > >> <https://www.linkedin.com/in/rmannibucau> | Tomitriber > > >> <http://www.tomitribe.com> > > >> > > >> 2015-07-27 19:16 GMT+02:00 Jean-Louis Monteiro < > > jlmonte...@tomitribe.com> > > >> : > > >> > > >>> Big +1 in simplifying the pool configuration. > > >>> > > >>> JLouis > > >>> > > >>> -- > > >>> Jean-Louis Monteiro > > >>> http://twitter.com/jlouismonteiro > > >>> http://www.tomitribe.com > > >>> > > >>> On Sun, Jul 26, 2015 at 6:36 AM, Romain Manni-Bucau < > > >>> rmannibu...@gmail.com> > > >>> wrote: > > >>> > > >>> > Hi guys, > > >>> > > > >>> > in tomee we support the ability to change the db/connection pool > > easily > > >>> > but IMO we have one issue with defaults. This is done thanks to > > >>> > "DataSourceCreator". > > >>> > > > >>> > When impl-ed it I tried to normalize some part of the config cause > it > > >>> was > > >>> > in between minors but now well get the version 7 I think we can > > change > > >>> it a > > >>> > bit and I d like to use the custom config for each pool > > >>> > and get rid of the implicit conversion. > > >>> > > > >>> > User issue: dbcp will use JdbcUrl where tomcat-jdbc will use Url - > > but > > >>> it > > >>> > would get reported in logs if the user didnt use the right one. > > >>> > > > >>> > Gain: no implicit value lost/misconfigured as today (today with our > > >>> > conversion logic some setters are ignored cause values are > rewritten > > >>> before > > >>> > the instantiation and we dont see it at all, common issue: > validation > > >>> is > > >>> > skipped). > > >>> > > > >>> > So here is the question: do we remove all the pool config from > > >>> > service-jar.xml + conversion from DataSourceCreators and only keep > > the > > >>> > connection (default datasource) and JTA (managed by default) > entries > > as > > >>> > provided defaults. > > >>> > > > >>> > Constraint to stay nice: still support Duration setter (instead of > > >>> 60000 > > >>> > for 1mn we can set as property value "1 minute"). > > >>> > > > >>> > +1 from me > > >>> > > > >>> > PS: if no big -1 next week (monday = 08/27) I'll do the change > > >>> > > > >>> > Romain Manni-Bucau > > >>> > @rmannibucau <https://twitter.com/rmannibucau> | Blog > > >>> > <http://rmannibucau.wordpress.com> | Github < > > >>> > https://github.com/rmannibucau> | > > >>> > LinkedIn <https://www.linkedin.com/in/rmannibucau> | Tomitriber > > >>> > <http://www.tomitribe.com> > > >>> > > > >>> > > >> > > >> > > >
