Uh oh!
There was an error while loading. Please reload this page.
[feature](nereids)support mark join - #30133
Conversation
starocean999
commented
Jan 19, 2024
run buildall |
| // specific language governing permissions and limitations | ||
| // under the License. | ||
| #include <gen_cpp/PlanNodes_types.h> |
There was a problem hiding this comment.
warning: 'gen_cpp/PlanNodes_types.h' file not found [clang-diagnostic-error]
#include<gen_cpp/PlanNodes_types.h>
^| std::vector<uint8_t>& visited, bool has_null_in_build_side); | ||
| template <bool with_other_conjuncts> | ||
| Status do_mark_join_conjuncts(Block* output_block, const size_t hash_table_bucket_size); |
There was a problem hiding this comment.
warning: parameter 'hash_table_bucket_size' is const-qualified in the function declaration; const-qualification of parameters only has an effect in function definitions [readability-avoid-const-params-in-decls]
| Statusdo_mark_join_conjuncts(Block*output_block, constsize_thash_table_bucket_size); | |
| Statusdo_mark_join_conjuncts(Block*output_block, size_thash_table_bucket_size); |
| template <int JoinOpType, typename Parent> | ||
| template <bool with_other_conjuncts> | ||
| Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts( |
There was a problem hiding this comment.
warning: function 'do_mark_join_conjuncts' exceeds recommended size/complexity thresholds [readability-function-size]
Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts(
^Additional context
be/src/vec/exec/join/process_hash_table_probe_impl.h:228: 85 lines including whitespace and comments (threshold 80)
Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts(
^| return Status::OK(); | ||
| } | ||
| Status VExprContext::execute_conjuncts(const VExprContextSPtrs& conjuncts, Block* block, |
There was a problem hiding this comment.
warning: method 'execute_conjuncts' can be made static [readability-convert-member-functions-to-static]
| Status VExprContext::execute_conjuncts(const VExprContextSPtrs& conjuncts, Block* block, | |
| staticStatus VExprContext::execute_conjuncts(const VExprContextSPtrs& conjuncts, Block* block, |
doris-robot
commented
Jan 19, 2024
TeamCity be ut coverage result: |
starocean999
commented
Jan 19, 2024
run buildall |
doris-robot
commented
Jan 19, 2024
TeamCity be ut coverage result: |
starocean999
commented
Jan 20, 2024
run buildall |
| template <int JoinOpType, typename Parent> | ||
| template <bool with_other_conjuncts> | ||
| Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts( |
There was a problem hiding this comment.
warning: function 'do_mark_join_conjuncts' exceeds recommended size/complexity thresholds [readability-function-size]
Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts(
^Additional context
be/src/vec/exec/join/process_hash_table_probe_impl.h:229: 82 lines including whitespace and comments (threshold 80)
Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts(
^mrhhsg
commented
Jan 20, 2024
run buildall |
doris-robot
commented
Jan 20, 2024
TeamCity be ut coverage result: |
starocean999
commented
Jan 21, 2024
run buildall |
doris-robot
commented
Jan 21, 2024
TeamCity be ut coverage result: |
doris-robot
commented
Jan 21, 2024
TPC-H: Total hot run time: 38586 ms |
doris-robot
commented
Jan 21, 2024
TPC-DS: Total hot run time: 176927 ms |
doris-robot
commented
Jan 21, 2024
ClickBench: Total hot run time: 30.2 s |
doris-robot
commented
Jan 21, 2024
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' |
mrhhsg
commented
Jan 21, 2024
run buildall |
| template <int JoinOpType, typename Parent> | ||
| template <bool with_other_conjuncts> | ||
| Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts( |
There was a problem hiding this comment.
warning: function 'do_mark_join_conjuncts' exceeds recommended size/complexity thresholds [readability-function-size]
Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts(
^Additional context
be/src/vec/exec/join/process_hash_table_probe_impl.h:229: 89 lines including whitespace and comments (threshold 80)
Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts(
^doris-robot
commented
Jan 21, 2024
TPC-H: Total hot run time: 39128 ms |
doris-robot
commented
Jan 21, 2024
TPC-DS: Total hot run time: 177193 ms |
doris-robot
commented
Jan 21, 2024
ClickBench: Total hot run time: 31.57 s |
doris-robot
commented
Jan 21, 2024
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' |
doris-robot
commented
Jan 21, 2024
TeamCity be ut coverage result: |
d4988f8 to
ff7c7faCompare| template <int JoinOpType, typename Parent> | ||
| template <bool with_other_conjuncts> | ||
| Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts( |
There was a problem hiding this comment.
warning: function 'do_mark_join_conjuncts' exceeds recommended size/complexity thresholds [readability-function-size]
Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts(
^Additional context
be/src/vec/exec/join/process_hash_table_probe_impl.h:233: 82 lines including whitespace and comments (threshold 80)
Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts(
^mrhhsg
commented
Jan 21, 2024
run buildall |
| template <int JoinOpType, typename Parent> | ||
| template <bool with_other_conjuncts> | ||
| Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts( |
There was a problem hiding this comment.
warning: function 'do_mark_join_conjuncts' exceeds recommended size/complexity thresholds [readability-function-size]
Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts(
^Additional context
be/src/vec/exec/join/process_hash_table_probe_impl.h:233: 89 lines including whitespace and comments (threshold 80)
Status ProcessHashTableProbe<JoinOpType, Parent>::do_mark_join_conjuncts(
^doris-robot
commented
Jan 21, 2024
TeamCity be ut coverage result: |
Co-authored-by: Jerry Hu <mrhhsg@gmail.com>
0358f42 to
66b6a55Comparestarocean999
commented
Jan 26, 2024
run buildall |
doris-robot
commented
Jan 26, 2024
TPC-H: Total hot run time: 39545 ms |
doris-robot
commented
Jan 26, 2024
TPC-DS: Total hot run time: 185839 ms |
doris-robot
commented
Jan 26, 2024
ClickBench: Total hot run time: 30.4 s |
doris-robot
commented
Jan 26, 2024
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' |
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
Co-authored-by: Jerry Hu <mrhhsg@gmail.com>
Co-authored-by: Jerry Hu <mrhhsg@gmail.com>
### What problem does this PR solve? Related PR: #30133 Problem Summary: The original PR eliminate null aware with a wrong way to produce wrong result under some cases. This PR remove this optimization and will take by it in a correct way in next PR.
### What problem does this PR solve? Related PR: #30133 Problem Summary: The original PR eliminate null aware with a wrong way to produce wrong result under some cases. This PR remove this optimization and will take by it in a correct way in next PR.
### What problem does this PR solve? Related PR: #30133 Problem Summary: The original PR eliminate null aware with a wrong way to produce wrong result under some cases. This PR remove this optimization and will take by it in a correct way in next PR.
Proposed changes
support mark join
be part is implement by Author: Jerry Hu <mrhhsg@gmail.com>
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...