Skip to content

Commit

Permalink
rename
Browse files Browse the repository at this point in the history
Signed-off-by: xufei <[email protected]>
  • Loading branch information
windtalker committed Apr 13, 2023
1 parent 0819900 commit 43d6f26
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions dbms/src/Interpreters/Join.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ Join::Join(
, kind(kind_)
, strictness(strictness_)
, original_strictness(strictness)
, may_block_expanded_after_join_block(mayBlockExpandedAfterJoinBlock(kind, strictness))
, may_probe_side_expanded_after_join(mayProbeSideExpandedAfterJoin(kind, strictness))
, key_names_left(key_names_left_)
, key_names_right(key_names_right_)
, build_concurrency(0)
Expand Down Expand Up @@ -982,7 +982,7 @@ Block Join::joinBlockHash(ProbeProcessInfo & probe_process_info) const
/// exit the while loop if
/// 1. probe_process_info.all_rows_joined_finish is true, which means all the rows in current block is processed
/// 2. the block may be expanded after join and result_rows exceeds the min_result_block_size
if (probe_process_info.all_rows_joined_finish || (may_block_expanded_after_join_block && result_rows >= probe_process_info.min_result_block_size))
if (probe_process_info.all_rows_joined_finish || (may_probe_side_expanded_after_join && result_rows >= probe_process_info.min_result_block_size))
break;
}
assert(!result_blocks.empty());
Expand Down
2 changes: 1 addition & 1 deletion dbms/src/Interpreters/Join.h
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ class Join
ASTTableJoin::Kind kind;
ASTTableJoin::Strictness strictness;
ASTTableJoin::Strictness original_strictness;
const bool may_block_expanded_after_join_block;
const bool may_probe_side_expanded_after_join;

/// Names of key columns (columns for equi-JOIN) in "left" table (in the order they appear in USING clause).
const Names key_names_left;
Expand Down
2 changes: 1 addition & 1 deletion dbms/src/Interpreters/JoinUtils.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ void computeDispatchHash(size_t rows,
}
}

bool mayBlockExpandedAfterJoinBlock(ASTTableJoin::Kind kind, ASTTableJoin::Strictness strictness)
bool mayProbeSideExpandedAfterJoin(ASTTableJoin::Kind kind, ASTTableJoin::Strictness strictness)
{
/// null aware semi/left semi/anti join never expand the block
if (isNullAwareSemiFamily(kind))
Expand Down
2 changes: 1 addition & 1 deletion dbms/src/Interpreters/JoinUtils.h
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ inline bool isNullAwareSemiFamily(ASTTableJoin::Kind kind)
|| kind == ASTTableJoin::Kind::NullAware_LeftSemi;
}

bool mayBlockExpandedAfterJoinBlock(ASTTableJoin::Kind kind, ASTTableJoin::Strictness strictness);
bool mayProbeSideExpandedAfterJoin(ASTTableJoin::Kind kind, ASTTableJoin::Strictness strictness);

struct ProbeProcessInfo
{
Expand Down

0 comments on commit 43d6f26

Please sign in to comment.