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

bump mysql version upto 8.0 #199

Merged
merged 5 commits into from
May 30, 2023
Merged

bump mysql version upto 8.0 #199

merged 5 commits into from
May 30, 2023

Conversation

windymelt
Copy link
Contributor

@windymelt windymelt commented May 26, 2023

Due to approaching of MySQL 5.7 EOL, we have to bump supported mysql version.

First, I updated version of MySQL of Dockerfile upto 8.0 for test.

Did

  • Bumped MySQL Version of Dockerfile for test upto 8.0 from 5.7
    • This does not affect fireworq binary
    • Test will use MySQL 8.0
  • Fixed deprecated syntax (VALUES inside ON DUPLICATE clause)
    • This causes breaking change

Breaking Change

  • New syntax (row aliasing) introduced -- this is only availavle since MySQL 8.0
    • It results dropping support of MySQL 5.7

Breaking Change retracted

To support smooth migration, we have to provide release supports both MySQL version. We can hold breaking change in further release. This PR sustains support for MySQL 5.7.

cf. #183

cf. https://dev.mysql.com/doc/refman/8.0/en/insert-on-duplicate.html

@windymelt windymelt marked this pull request as ready for review May 26, 2023 06:38
@windymelt
Copy link
Contributor Author

Benchmark Test

I tested Fireworq score between MySQL 5.7 vs MySQL 8.0 (AWS RDS) using fireworq/benchworker.

Score got worse, but we can recover it using newer RDS instance. MySQL 8.0 on m6g.large performs as good as MySQL 5.7 on m5.large.

image

Comparing MySQL 5.7 on m5.large and MySQL 8.0 on m6g.large:

image

Copy link
Member

@tarao tarao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@windymelt LGTM

@windymelt windymelt merged commit e1ab9e4 into master May 30, 2023
2 checks passed
@windymelt windymelt deleted the bump-ci-mysql-version-8 branch May 30, 2023 08:49
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

2 participants