Skip to content

Minor changes to support SET DEFAULT FK referential action#9121

Merged
fulghum merged 2 commits intomainfrom
fulghum/fk_ref_actions
Apr 18, 2025
Merged

Minor changes to support SET DEFAULT FK referential action#9121
fulghum merged 2 commits intomainfrom
fulghum/fk_ref_actions

Conversation

@fulghum
Copy link
Copy Markdown
Contributor

@fulghum fulghum commented Apr 16, 2025

MySQL parses the ON UPDATE/DELETE SET DEFAULT syntax, but does not actually support executing it. These minor changes allow the SET DEFAULT referential action to be used from Doltgres, but do not enable it for Dolt, to preserve MySQL compatibility.

Depends on: dolthub/go-mysql-server#2944

fulghum added 2 commits April 18, 2025 10:25
…t SET DEFAULT. Dolt/GMS will still validate this in an analyzer rule to preserve MySQL compatibility.
@fulghum fulghum force-pushed the fulghum/fk_ref_actions branch from e6fc9e3 to 0f045d8 Compare April 18, 2025 17:30
@dolthub dolthub deleted a comment from coffeegoddd Apr 18, 2025
@coffeegoddd
Copy link
Copy Markdown
Contributor

@fulghum DOLT

comparing_percentages
100.000000 to 100.000000
version result total
0f045d8 ok 5937457
version total_tests
0f045d8 5937457
correctness_percentage
100.0

@dolthub dolthub deleted a comment from coffeegoddd Apr 18, 2025
Copy link
Copy Markdown
Member

@zachmu zachmu left a comment

Choose a reason for hiding this comment

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

LGTM

@fulghum fulghum merged commit e563198 into main Apr 18, 2025
21 checks passed
@tbantle22 tbantle22 deleted the fulghum/fk_ref_actions branch May 28, 2025 21:37
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