Skip to content

Do not allow sort-based joins between text and number type columns#3423

Merged
angelamayxie merged 5 commits intomainfrom
angela/sortorder
Feb 10, 2026
Merged

Do not allow sort-based joins between text and number type columns#3423
angelamayxie merged 5 commits intomainfrom
angela/sortorder

Conversation

@angelamayxie
Copy link
Copy Markdown
Contributor

@angelamayxie angelamayxie commented Feb 7, 2026

fixes dolthub/dolt#10435

Disables merge and range heap joins between text and number type columns

@angelamayxie angelamayxie changed the title Do not allow sort-based merges between text and number type columns Do not allow sort-based joins between text and number type columns Feb 10, 2026
@angelamayxie angelamayxie requested a review from jycor February 10, 2026 20:40
@angelamayxie angelamayxie marked this pull request as ready for review February 10, 2026 20:40
Copy link
Copy Markdown
Contributor

@jycor jycor left a comment

Choose a reason for hiding this comment

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

LGTM

@angelamayxie angelamayxie merged commit 4482301 into main Feb 10, 2026
9 of 11 checks passed
@angelamayxie angelamayxie deleted the angela/sortorder branch February 10, 2026 21:02
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.

Disable RangeHeapJoin with string and number columns

2 participants