Skip to content

Fix explicit broadcast join bug - #4424

Merged
morningman merged 4 commits into
apache:masterfrom
wyb:broadcast_join_explicit_bug
Aug 25, 2020
Merged

Fix explicit broadcast join bug#4424
morningman merged 4 commits into
apache:masterfrom
wyb:broadcast_join_explicit_bug

Conversation

@wyb

@wybwyb commented Aug 21, 2020

Copy link
Copy Markdown
Contributor

Use broadcast join when users specify explicitly [BROADCAST] in queries.

kangkaisen
kangkaisen reviewed Aug 21, 2020
View reviewed changes
Comment threadfe/fe-core/src/main/java/org/apache/doris/planner/DistributedPlanner.java Outdated
kangkaisen
kangkaisen reviewed Aug 21, 2020
View reviewed changes
Comment threadfe/fe-core/src/main/java/org/apache/doris/planner/DistributedPlanner.java Outdated
kangkaisen
kangkaisen approved these changes Aug 21, 2020
View reviewed changes

@kangkaisenkangkaisen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1

@morningmanmorningman added approved Indicates a PR has been approved by one committer. area/planner Issues or PRs related to the query planner labels Aug 25, 2020
@morningman
morningman merged commit 6912279 into apache:masterAug 25, 2020
@yangzhgyangzhg mentioned this pull request Feb 9, 2021
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kangkaisenkangkaisenkangkaisen approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wyb@kangkaisen@morningman