Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(query): vectorized hash join #13103

Merged
merged 2 commits into from
Oct 8, 2023

Conversation

Dousir9
Copy link
Member

@Dousir9 Dousir9 commented Oct 7, 2023

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

Vectorized hash join, we now perform probe hash table and key matching separately, this allows us to take advantage of SIMD instructions, after testing partial SQL of TPC-H Q18(select l_partkey, o_custkey from lineitem, orders where l_orderkey = o_orderkey ignore_result;), the performance of hash join has been improved by ~10%(6.6s -> 5.9s).

  • Closes #issue

This change is Reviewable

@vercel
Copy link

vercel bot commented Oct 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
databend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 7, 2023 11:06am

@Dousir9 Dousir9 requested review from xudong963, sundy-li and zhang2014 and removed request for xudong963 and sundy-li October 7, 2023 11:03
@Dousir9 Dousir9 marked this pull request as ready for review October 7, 2023 11:04
@github-actions github-actions bot added the pr-refactor this PR changes the code base without new features or bugfix label Oct 7, 2023
@Dousir9 Dousir9 added ci-benchmark Benchmark: run all test and removed pr-refactor this PR changes the code base without new features or bugfix labels Oct 7, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2023

Docker Image for PR

  • tag: pr-13103-d3d43af

note: this image tag is only available for internal use,
please check the internal doc for more details.

@Dousir9 Dousir9 added ci-benchmark Benchmark: run all test and removed ci-benchmark Benchmark: run all test labels Oct 7, 2023
@databendlabs databendlabs deleted a comment from github-actions bot Oct 7, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2023

Docker Image for PR

  • tag: pr-13103-8633d98

note: this image tag is only available for internal use,
please check the internal doc for more details.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2023

@BohuTANG BohuTANG added ci-benchmark Benchmark: run all test and removed ci-benchmark Benchmark: run all test labels Oct 8, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2023

Docker Image for PR

  • tag: pr-13103-543bd09

note: this image tag is only available for internal use,
please check the internal doc for more details.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2023

@BohuTANG BohuTANG merged commit 8b79384 into databendlabs:main Oct 8, 2023
90 checks passed
andylokandy pushed a commit to andylokandy/databend that referenced this pull request Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-benchmark Benchmark: run all test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants