Skip to content

[optimize] init join hash table size by rows before hash table building - #9732

Closed
englefly wants to merge 2 commits into
apache:masterfrom
englefly:join_hash_table_resize
Closed

[optimize] init join hash table size by rows before hash table building#9732
englefly wants to merge 2 commits into
apache:masterfrom
englefly:join_hash_table_resize

Conversation

@englefly

Copy link
Copy Markdown
Contributor

Proposed changes

Initialize hash table size by the tuple number instead of fixed number 1024 to reduce BuildTableExpanseTime.
test result:
After initialize table size, the total build time decreased by 8.9% on tpch 10G,
test sql:
select count(*) from lineitem join orders on l_orderkey = o_orderkey

Problem Summary:

Describe the overview of changes.

Checklist(Required)

  1. Does it affect the original behavior: (Yes/No/I Don't know)
  2. Has unit tests been added: (Yes/No/No Need)
  3. Has document been added or modified: (Yes/No/No Need)
  4. Does it need to update dependencies: (Yes/No)
  5. Are there any changes that cannot be rolled back: (Yes/No)

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@yiguolei

Copy link
Copy Markdown
Contributor

Duplicate with #9735

@englefly
englefly deleted the join_hash_table_resize branch August 5, 2022 16:48
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@englefly@yiguolei