edulify / play-hikaricp.edulify.com

HikariCP Plugin for Playframework 2.2.x and 2.3.x
http://edulify.github.io/play-hikaricp.edulify.com/
Apache License 2.0
82 stars 35 forks source link

Get rid of "jdbcConnectionTest" warning when using Play configuration #40

Closed julien-lafont closed 9 years ago

julien-lafont commented 9 years ago

When HikariCP is configured through play application.conf, a warning is always displayed in the console.

[warn] c.z.h.HikariConfig - The jdbcConnectionTest property is now deprecated, see the
documentation for connectionTestQuery

This warning is triggered when the jdbc4ConnectionTest property is setted

It's not possible to remove this warning by tweaking play configuration, due to this line which always set this property.

megazord commented 9 years ago

Yes, you are right.

I have plans to remove all configuration options in favor of have Hikari configured directly in application.conf file. In other words, we won't have this translation between Play properties and Hikari. I expect to work on that the next days.

julien-lafont commented 9 years ago

Yes it seems a good plan. It's easier to inject a custom config in tests with the Play configuration than Hikari properties.

megazord commented 9 years ago

Fixed at master branch.