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

CREATE, DROP, ALTER TABLE causes MDL BF-BF conflict #463

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

Conversation

hemantdangi-gc
Copy link

Issue:

During DROP TABLE execution, if parent tables are opened and MDL locks for those tables are acquired, then concurrent operations to those table could end up in MDL conflict.

For CREATE and ALTER TABLE foreign keys of immediate parent table are getting appended in certiification, but if parent table also has foreign key and if those refrenced tables are opened and MDL locks for those tables are acquired, then concurrent operations to those tables could also end up in MDL conflict.

Solution:

To avoid MDL BF-BF conflicts, append certification keys for all parent's tables so that potentially conflicting operation are not applied in parallel with CREATE, DROP, ALTER TABLE command.

Issue:
------
During DROP TABLE execution, if parent tables are opened and MDL locks
for those tables are acquired, then concurrent operations to those
table could end up in MDL conflict.

For CREATE and ALTER TABLE foreign keys of immediate parent table are
getting appended in certiification, but if parent table also has foreign
key and if those refrenced tables are opened and MDL locks for those
tables are acquired, then concurrent operations to those tables could also
end up in MDL conflict.

Solution:
---------
To avoid MDL BF-BF conflicts, append certification keys for all parent's
tables so that potentially conflicting operation are not applied in
parallel with CREATE, DROP, ALTER TABLE command.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants