Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bumped HikariCP version to 2.4.0 #5003

Closed
wants to merge 1 commit into from

Conversation

dotta
Copy link
Contributor

@dotta dotta commented Aug 17, 2015

Also, note that HikariCP 2.4.x stream will be binary compatible with 2.4.0 as
mentioned in
https://groups.google.com/forum/#!topic/play-framework-dev/iqKgOqa3McM

do NOT backport :-)

@marcospereira
Copy link
Member

@dotta I've closed and reopen the PR just to trigger a new build.

@jroper
Copy link
Member

jroper commented Aug 18, 2015

This is failing due to a compile error.

@marcospereira
Copy link
Member

Here is the error:

[error] /home/travis/build/playframework/playframework/framework/src/play-jdbc/src/test/scala/play/api/db/HikariCPConfigSpec.scala:82: type mismatch;
[error]  found   : org.specs2.matcher.Matcher[scala.Boolean]
[error]  required: org.specs2.matcher.Matcher[java.lang.Boolean]
[error]         new HikariCPConfig(dbConfig, reference).toHikariConfig.isReadOnly must beFalse
[error]                                                                                ^
[error] one error found

dotta referenced this pull request in brettwooldridge/HikariCP Aug 18, 2015
Had to insert a type ascription in a couple of tests to force the conversion of
a `java.lang.Boolean` into a `scala.Boolean`. The change was needed because the
type of `HikariConfig.isReadOnly` was changed from a primitive boolean into a
boxed one (see
brettwooldridge/HikariCP@b18c786#commitcomment-12755016)

Also, note that HikariCP 2.4.x stream will be binary compatible with 2.4.0 as
mentioned in
https://groups.google.com/forum/#!topic/play-framework-dev/iqKgOqa3McM
@dotta
Copy link
Contributor Author

dotta commented Aug 19, 2015

I'm closing this as there will be a hikaricp 2.4.1 release soon (and some of the changes in this PR won't be needed anymore). For context see brettwooldridge/HikariCP@b18c786#commitcomment-12777354

@dotta dotta closed this Aug 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants