Skip to content

Conversation

@wutiangan
Copy link
Contributor

fix #4149

add sessionVariable "prefer_broadcast_join" to choose broadcastjoin first when cardinality cannot be estimated
the sessionVariable can set local or global. the default value is true, prefer broadcast join first where the two cost is equal.

kangkaisen
kangkaisen previously approved these changes Jul 23, 2020
Copy link
Contributor

@kangkaisen kangkaisen left a comment

Choose a reason for hiding this comment

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

+1

morningman
morningman previously approved these changes Jul 23, 2020
Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

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

LGTM

@morningman morningman added approved Indicates a PR has been approved by one committer. area/planner Issues or PRs related to the query planner labels Jul 23, 2020
Copy link
Contributor

@kangkaisen kangkaisen left a comment

Choose a reason for hiding this comment

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

+1

@kangkaisen kangkaisen merged commit 59676a1 into apache:master Jul 29, 2020
morningman pushed a commit to morningman/doris that referenced this pull request Jul 30, 2020
…n cardinality cannot be estimated (apache#4150)

Change-Id: I92f458e353657fe2410430cb8e6c0b60a24441cb
bobhan1 pushed a commit to bobhan1/doris that referenced this pull request Jul 15, 2025
liaoxin01 added a commit to liaoxin01/doris that referenced this pull request Sep 10, 2025
liaoxin01 added a commit to liaoxin01/doris that referenced this pull request Sep 10, 2025
csun5285 pushed a commit to csun5285/doris that referenced this pull request Sep 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. area/planner Issues or PRs related to the query planner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] choose broadcastjoin first when cardinality cannot be estimated

3 participants