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

MDEV-33175 applier FK check failure retrying #385

Open
wants to merge 1 commit into
base: 10.4
Choose a base branch
from

Conversation

sjaakola
Copy link

@sjaakola sjaakola commented Jan 4, 2024

Implementation of foreign key constraint check retrying for replication appliers

Foreign key constraint checks may occaionally fail even though the constraints are not violated. There have been cases where e.g. the same transaction inserts in the parent table, and the next insert into child table fails in foreign key checks. The underlying reason for these failures is not known.

This change adds retrying of constraint check in InnoDB. The number of retries is by default 1, and the number of retries can be controlled by a new system variable
wsrep-applier-FK-failure-retries`. If the constraint check fails despite retries, the final retry prints out a warning with an error code and InnoDB system monitor output for further troubleshooting.

Implementation of foreign key constraint check retrying
for replication appliers

Foreign key constraint checks may occaionally fail even though
the constraints are not violated. There have been cases where e.g.
the same transaction inserts in the parent table, and the next
insert into child table fails in foreign key checks. The underlying
reason for these failures is not known.

This change adds retrying of constraint check in InnoDB. The
number of retries is by default 1, and the number of retries
can be controlled by a new system variable
wsrep-applier-FK-failure-retries`. If the constraint check fails
despite retries, the final retry prints out a warning with an
error code and InnoDB system monitor output for further troubleshooting.

Co-authored-by: Teemu Ollakka <[email protected]>
@sjaakola
Copy link
Author

sjaakola commented Jan 8, 2024

test_pr

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