Skip to content

Conversation

@EchoLee5
Copy link
Contributor

@EchoLee5 EchoLee5 commented Dec 4, 2023

Current binlog offset add serverId, fix incremental-snapshot scan data lose
link #2794

Copy link
Contributor

@ruanhang1993 ruanhang1993 left a comment

Choose a reason for hiding this comment

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

@EchoLee5 Thanks for the fix. Could you add some tests to cover this bug?
I think we could start a mysql container without gtid-mode = on to reproduce this bug.

@ruanhang1993
Copy link
Contributor

The problem will occur when the gtid mode is closed and some data changes happen in the snapshot phase.

@EchoLee5
Copy link
Contributor Author

EchoLee5 commented Jan 9, 2024

The problem will occur when the gtid mode is closed and some data changes happen in the snapshot phase.

Sorry for being late, I have added unit tests, please review them for me, thank you!

@yuxiqian
Copy link
Member

Hi @EchoLee5, could you please rebase this PR with latest master branch before it could be merged?

cc @ruanhang1993

@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity for 60 days. It will be closed in 30 days if no further activity occurs.

@github-actions github-actions bot added the Stale label Jul 17, 2024
@github-actions github-actions bot removed the Stale label Jul 26, 2024
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity for 60 days. It will be closed in 30 days if no further activity occurs.

@github-actions github-actions bot added Stale and removed Stale labels Sep 24, 2024
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity for 60 days. It will be closed in 30 days if no further activity occurs.

@github-actions github-actions bot added the Stale label Nov 24, 2024
@github-actions
Copy link

This pull request has been closed because it has not had recent activity. You could reopen it if you try to continue your work, and anyone who are interested in it are encouraged to continue work on this pull request.

@github-actions github-actions bot closed this Dec 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants