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

Update mysql-connector to 8.0.13 #1495

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

col-panic
Copy link
Contributor

As mentioned in #1494 - I need to be able to connect to newer mysql databases, which is not possible given the old version of mysql.

I built the patched version locally and applied the overlay on it.

The resulting product runs as required, now delivering some (cosmetic) warnings on startup. Removing those warnings should required some source changes.

Loading class `com.mysql.jdbc.Driver'. This is deprecated. The new driver class is `com.mysql.cj.jdbc.Driver'. The driver is automatically registered via the SPI and manual loading of the driver class is generally unnecessary.
...
[EL Info]: 2019-07-09 09:52:49.384--ServerSession(2141526452)--EclipseLink, version: Eclipse Persistence Services - 2.7.4.v20190115-ad5b7c6b2a
[EL Info]: connection: 2019-07-09 09:52:49.917--ServerSession(2141526452)--/file:/private/var/folders/rq/bwqzcstj45lbsxj83v189jg40000gn/T/jetty-0.0.0.0-8380-openid.war-_openid-any-3436388020107941948.dir/webapp/WEB-INF/lib/openid-connect-common-1.3.4-SNAPSHOT.jar_defaultPersistenceUnit login successful
[EL Info]: 2019-07-09 09:52:49.999--ServerSession(1782136855)--EclipseLink, version: Eclipse Persistence Services - 2.7.4.v20190115-ad5b7c6b2a
WARN : com.zaxxer.hikari.util.DriverDataSource - Registered driver with driverClassName=com.mysql.jdbc.Driver was not found, trying direct instantiation.
[EL Info]: connection: 2019-07-09 09:52:50.158--ServerSession(1782136855)--/bundleresource://60.fwk1403704789/_elexisPersistenceUnit login successful

@codecov-io
Copy link

Codecov Report

Merging #1495 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1495   +/-   ##
=========================================
  Coverage     25.21%   25.21%           
  Complexity      912      912           
=========================================
  Files           209      209           
  Lines         11689    11689           
  Branches       2116     2116           
=========================================
  Hits           2947     2947           
  Misses         8227     8227           
  Partials        515      515

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc6bd4b...81c1e0e. Read the comment docs.

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.

2 participants