Skip to content

fix panic for empty table names, now a normal error#10450

Merged
zachmu merged 2 commits intomainfrom
zachmu/table-name-panic
Feb 12, 2026
Merged

fix panic for empty table names, now a normal error#10450
zachmu merged 2 commits intomainfrom
zachmu/table-name-panic

Conversation

@zachmu
Copy link
Copy Markdown
Member

@zachmu zachmu commented Feb 7, 2026

No description provided.

@zachmu zachmu requested a review from reltuk February 7, 2026 02:07
@coffeegoddd
Copy link
Copy Markdown
Contributor

@zachmu DOLT

comparing_percentages
100.000000 to 100.000000
version result total
71fd4c2 ok 5937471
version total_tests
71fd4c2 5937471
correctness_percentage
100.0

@coffeegoddd
Copy link
Copy Markdown
Contributor

@coffeegoddd DOLT

comparing_percentages
100.000000 to 100.000000
version result total
7ef81ec ok 5937471
version total_tests
7ef81ec 5937471
correctness_percentage
100.0

Copy link
Copy Markdown
Contributor

@macneale4 macneale4 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@macneale4 macneale4 left a comment

Choose a reason for hiding this comment

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

LGTM, for reals

@zachmu zachmu merged commit 7c58d97 into main Feb 12, 2026
24 of 26 checks passed
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