[WIP][pipelineX](local shuffle) Use local shuffle to optimize BHJ #27823
Conversation
Gabriel39
changed the title
|
run buildall |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| return _sub_plan_query_statistics_recvr; | ||
| } | ||
|
|
||
| bool need_to_local_shuffle() const override { |
There was a problem hiding this comment.
warning: method 'need_to_local_shuffle' can be made static [readability-convert-member-functions-to-static]
| bool need_to_local_shuffle() const override { | |
| static bool need_to_local_shuffle() override { |
| return Status::OK(); | ||
| } | ||
|
|
||
| Status PipelineXFragmentContext::_plan_local_shuffle() { |
There was a problem hiding this comment.
warning: method '_plan_local_shuffle' can be made static [readability-convert-member-functions-to-static]
be/src/pipeline/pipeline_x/pipeline_x_fragment_context.h:155:
- Status _plan_local_shuffle();
+ static Status _plan_local_shuffle();
|
(From new machine)TeamCity pipeline, clickbench performance test result: |
|
PR approved by at least one committer and no changes requested. |
github-actions
bot
added
approved
|
PR approved by anyone and no changes requested. |
TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' |
4 participants
Proposed changes
Issue Number: close #xxx
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...