Skip to content

Conversation

@comphead
Copy link
Contributor

@comphead comphead commented Jun 6, 2024

Which issue does this PR close?

Closes #.
Followup on #10747
Related #9846

Rationale for this change

By default benches run with hash join algorithm, this PR introduces new env variable PREFER_HASH_JOIN to give the user a switch for running benchmarks with SMJ as well.

What changes are included in this PR?

Removed old SMJ benches code, added env variable and fixed the doc

Are these changes tested?

Yes manually

Are there any user-facing changes?

No

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.

looks good to me -- thanks @comphead

@alamb alamb merged commit 7c91627 into apache:main Jun 6, 2024
findepi pushed a commit to findepi/datafusion that referenced this pull request Jul 16, 2024
* Fix: Sort Merge Join crashes on TPCH Q21

* Fix LeftAnti SMJ join when the join filter is set

* rm dbg

* Bench: Add `PREFER_HASH_JOIN` env variable

* Bench: Add `PREFER_HASH_JOIN` env variable
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.

2 participants