Skip to content

[branch-53] fix: HashJoin panic with String dictionary keys (don't flatten keys) (#20505)#20791

Merged
alamb merged 1 commit intoapache:branch-53from
hareshkh:b53-20505
Mar 10, 2026
Merged

[branch-53] fix: HashJoin panic with String dictionary keys (don't flatten keys) (#20505)#20791
alamb merged 1 commit intoapache:branch-53from
hareshkh:b53-20505

Conversation

@hareshkh
Copy link
Contributor

@hareshkh hareshkh commented Mar 8, 2026

apache#20505)

## Which issue does this PR close?

- Fixes apache#20696
- Follow on to apache#20441

## Rationale for this change


apache#20441 (review)
fixes the special case DictionaryArray handling in Joins. However, I
don't think we need to special case DictionaryArrays at all

## What changes are included in this PR?

1. Remove the special case dictionary handling

## Are these changes tested?

Yes by CI
## Are there any user-facing changes?

No (though maybe some queries get faster)
@hareshkh
Copy link
Contributor Author

@alamb @comphead : Can you please take a pass when you have some time? We would need this as a bugfix to bring over everything from 52.3.0 to 53.0.0?

Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

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

Thanks @hareshkh -- I restarted the CI for the failed doc build too which seems like an infrastructure issue

@alamb alamb merged commit 7698fdc into apache:branch-53 Mar 10, 2026
55 of 56 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

physical-plan Changes to the physical-plan crate sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants