Skip to content

[embulk-input-mysql] Use Latest 5.1.49 JDBC Driver #250

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

Closed
wants to merge 1 commit into from

Conversation

y-ken
Copy link
Contributor

@y-ken y-ken commented Mar 15, 2023

Hi @hiroyuki-sato

I know we would better to switch Connector/J 8.X, prior to that, how about to use latest current JDBC minor version?

Since I'm facing javax.net.ssl.SSLException: closing inbound before receiving peer's close_notify error with following combination, I'm wondering whether I need to use later version of JDBC Driver.

  • AWS RDS MySQL 5.7.36
  • embulk v0.9.17
  • embulk-input-mysql (0.10.0)
  • JDBC Driver = /home/digdag/.embulk/lib/gems/gems/embulk-input-mysql-0.10.0/default_jdbc_driver/mysql-connector-java-5.1.44.jar

According to this article, it recommend to use more later version of JDBC.
https://tech.omablo.com/archives/1042
https://takeda-san.hatenablog.com/entry/2019/01/24/004526

From release note, there are mention about SSL related improvent.
https://docs.oracle.com/cd/E17952_01/connector-j-5.1-relnotes-en/news-5-1-49.html

Therefore I feel better to use latest of 5.1.x series of JDBC Driver as default. that driver are released on Apr 20, 2020. it was 3 years ago.

@y-ken
Copy link
Contributor Author

y-ken commented Mar 15, 2023

Do I better to follow this way?
#237 (comment)

@dmikurube
Copy link
Member

Yeah, if you're in a hurry for the issue, please follow: #237 (comment)

We're thinking about the plan to upgrade. But I guess things would happen after the Embulk core v0.11 is out.

@y-ken
Copy link
Contributor Author

y-ken commented Mar 16, 2023

Gotcha

@y-ken y-ken closed this Mar 16, 2023
@y-ken y-ken deleted the patch-2 branch March 16, 2023 14:57
@y-ken
Copy link
Contributor Author

y-ken commented Mar 16, 2023

just in case, which file is expected to specify driver_path?

  • mysql-connector-java-5.1.49.jar
  • mysql-connector-java-5.1.49-bin.jar

@dmikurube
Copy link
Member

I'm not sure. We use mysql-connector from Maven Central, not mysql.com.

@y-ken
Copy link
Contributor Author

y-ken commented Mar 21, 2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants