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

on multithreaded slaves when gtid_mode is off the slave status could be inconsistence #144

Closed
liuguoshun opened this issue Jul 17, 2018 · 5 comments · Fixed by #417
Closed

Comments

@liuguoshun
Copy link

liuguoshun commented Jul 17, 2018

Please refer doc about "gap-free low-watermark position".
https://dev.mysql.com/doc/refman/5.7/en/replication-features-transaction-inconsistencies.html

on multithreaded slaves when gtid_mode is off ,need print an error info or abort the backup.

@liuguoshun liuguoshun changed the title on multithreaded slaves when gtid_mode is off the master status could be inconsistence on multithreaded slaves when gtid_mode is off the slave status could be inconsistence Jul 17, 2018
@davidducos davidducos added the bug label Apr 16, 2021
@davidducos davidducos added this to the Release 0.10.11 milestone Jul 19, 2021
@davidducos
Copy link
Member

Can you explain a bit more about this scenario? as from what I understood, you are trying to take a backup from a multithreaded database and the position saved in metadata file might be no correct. Am i right?

@liuguoshun
Copy link
Author

Yes, mydumper use "FLUSH TABLES WITH READ LOCK" and "SHOW SLAVE STATUS" to get the Exec_master_log_pos.
Pos of "SLAVE STATUS" maybe not correct in metadata file, when backup multithreaded replica database.

@davidducos
Copy link
Member

Ok, we are going to show a warning so users can disregard that info and start the replication from the replica where they are taking the backup instead of the source of the replication.

@davidducos davidducos linked a pull request Sep 29, 2021 that will close this issue
@davidducos
Copy link
Member

@liuguoshun please, let me know if #417 is ok, so we can merge to master.

Thank you!!

@liuguoshun
Copy link
Author

Great! :-)

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

Successfully merging a pull request may close this issue.

2 participants