Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

Avoiding infinite ancestry on circular replication #673

Merged
merged 2 commits into from
Nov 1, 2018

Conversation

shlomi-noach
Copy link
Collaborator

@shlomi-noach shlomi-noach commented Oct 29, 2018

Fixes #672

This PR adds a stop-condition to prevent infinitely growing ancestry_uuid in a circular replication situation.

cc @cohenjo

@cohenjo
Copy link

cohenjo commented Oct 29, 2018

Hi - Just finished testing - seems to resolve the issue - we don't see the infinite ancestry UUID list.
Thank you very much for the quick response 👍

…de each other). Errant GTID ignores transactions injected by a server in a co-master setup
@shlomi-noach
Copy link
Collaborator Author

This is now feature complete.

ancestry_uuid: both co-masters include each other in ancestry
gtid_errant: a co-master's own transactions are ignored when evaluating errant transactions.

@cohenjo
Copy link

cohenjo commented Oct 30, 2018

tested both - it cleared our errant transaction marking from the co-masters.
Thanks 👍

@shlomi-noach shlomi-noach temporarily deployed to production/mysql_cluster=concertmaster October 31, 2018 06:21 Inactive
@shlomi-noach shlomi-noach merged commit b29a377 into master Nov 1, 2018
@shlomi-noach shlomi-noach deleted the ancestry-uuid-co-masters branch November 1, 2018 06:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants