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

Add MySQL 8.4 Support #1494

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

chen-anders
Copy link

@chen-anders chen-anders commented Jan 27, 2025

Related issue: #1493

Description

MySQL 8.4 deprecates several old-style master/slave commands. Unfortunately, we had upgraded our DB to 8.4.3 and then found out after the fact that gh-ost broke, so this PR aims to introduce compatibility across 8.4+.

TODO: Update bash scripts used in tests to grep for the correct strings related to master/slave status

In case this PR introduced Go code changes:

  • contributed code is using same conventions as original code
  • script/cibuild returns with no formatting errors, build errors or unit test errors.

I wasn't able to get a minimal mysql 8.4 tarball that would actually pass the CI tests with dbdeployer but did manage to get the docker version running and passing: https://github.com/chen-anders/gh-ost/actions/runs/13026574857

@chen-anders chen-anders force-pushed the anders/mysql-8.4-support branch 2 times, most recently from 14607eb to 85b26b7 Compare January 28, 2025 08:02
@chen-anders chen-anders force-pushed the anders/mysql-8.4-support branch from 1b3f7f4 to 769767f Compare January 28, 2025 08:49
@chen-anders chen-anders force-pushed the anders/mysql-8.4-support branch from ffc60f3 to e2114df Compare January 29, 2025 15:51
@chen-anders chen-anders force-pushed the anders/mysql-8.4-support branch from e2114df to 7142b2b Compare January 29, 2025 15:52
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.

1 participant