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>

Reply via email to