Skip to content

test: re-enable merge join test with forced collisions#12276

Merged
alamb merged 1 commit intoapache:mainfrom
korowa:smj-hash-collisions
Sep 3, 2024
Merged

test: re-enable merge join test with forced collisions#12276
alamb merged 1 commit intoapache:mainfrom
korowa:smj-hash-collisions

Conversation

@korowa
Copy link
Contributor

@korowa korowa commented Sep 1, 2024

Which issue does this PR close?

Closes #11659.

Rationale for this change

Failure is not reproducing neither locally nor in CI -- seems like the test can be uncommented.

What changes are included in this PR?

The ignored test enabled again.

Are these changes tested?

Are there any user-facing changes?

@github-actions github-actions bot added the sqllogictest SQL Logic Tests (.slt) label Sep 1, 2024
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 @korowa -- let's try this and if we see it failing again we can investigate more

@alamb alamb merged commit 7ca9810 into apache:main Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

equijoin_full_and_condition_from_both slt test fails with force_hash_collisions

2 participants