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

Persist table comment from create table #7441

Merged
merged 11 commits into from
Feb 1, 2024
Merged

Conversation

fulghum
Copy link
Contributor

@fulghum fulghum commented Jan 31, 2024

We already parse table comments, but they were not persisted anywhere. This change adds support for persisting table comments so that they can be round-tripped between create table and show create table.

In addition to the follow ups in the GMS layer (mentioned in dolthub/go-mysql-server#2307), there is one follow up in the dolt layer to update Dolt's merge logic to merge table comment changes.

Depends on dolthub/go-mysql-server#2307

Related to #7416

@coffeegoddd
Copy link
Contributor

@fulghum DOLT

comparing_percentages
99.999074 to 99.994240
version result total
7894356 not ok 342
7894356 ok 5937115
version total_tests
7894356 5937457
correctness_percentage
99.99424

@coffeegoddd
Copy link
Contributor

@fulghum DOLT

comparing_percentages
99.999074 to 99.999074
version result total
95048e7 not ok 55
95048e7 ok 5937402
version total_tests
95048e7 5937457
correctness_percentage
99.999074

@fulghum fulghum requested a review from zachmu February 1, 2024 00:45
Copy link
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 88b475c into main Feb 1, 2024
21 checks passed
@fulghum fulghum deleted the fulghum/table-comments branch February 1, 2024 22:32
@coffeegoddd
Copy link
Contributor

@fulghum DOLT

comparing_percentages
99.999074 to 99.999074
version result total
89355f4 not ok 55
89355f4 ok 5937402
version total_tests
89355f4 5937457
correctness_percentage
99.999074

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