Skip to content

Feature/changed mysql replication command #53

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 4 commits into from

Conversation

soulee-dev
Copy link

Issue

Issue: #10

Problem

Since the SHOW SLAVE HOSTS statement was deprecated from MySQL 8.0.22[1], the MySQL variable slave_uuid is also be deprecated.

Solution

To ensure version compatibility, both the replica_uuid and slave_uuid variables will be set to the MySQL Server.

This approach aligns with what was done in go-mysql[2].

Additional Consideration

It would be prudent to change the name of the slave_uuid attribute to server_uuid, as it encapsulates the meanings of both slave_uuid and replica_uuid.


[1] SHOW SLAVE HOSTS | SHOW REPLICAS Statement (MySQL Docs)
[2] Set slave_uuid and replica_uuid (go-mysql-org/go-mysql)

Copy link

@dongwook-chan dongwook-chan left a comment

Choose a reason for hiding this comment

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

분기타지 않았고, go-mysql 레퍼런스가 존재하기 때문에 좋은 솔루션인 것 같습니다.
keyword 대문화해주신 것도 너무 좋아요!
별도 이슈로 keyword 모두 대문자화하는 해보는 것도 좋을 것 같네요!

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

Successfully merging this pull request may close these issues.

3 participants