Skip to content

Commit

Permalink
revert squashing_transform in HashJoinProbeBlockInputStream
Browse files Browse the repository at this point in the history
----
merge output blocks if need in hash join #6529
816b8d5
----
  • Loading branch information
solotzg committed Feb 2, 2023
1 parent a69dfbc commit 2b1e538
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 14 deletions.
14 changes: 1 addition & 13 deletions dbms/src/DataStreams/HashJoinProbeBlockInputStream.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ HashJoinProbeBlockInputStream::HashJoinProbeBlockInputStream(
, join(join_)
, probe_index(probe_index_)
, probe_process_info(max_block_size)
, squashing_transform(max_block_size)
{
children.push_back(input);

Expand Down Expand Up @@ -85,18 +84,7 @@ Block HashJoinProbeBlockInputStream::readImpl()
{
try
{
// if join finished, return {} directly.
if (squashing_transform.isJoinFinished())
{
return Block{};
}

while (squashing_transform.needAppendBlock())
{
Block result_block = getOutputBlock();
squashing_transform.appendBlock(result_block);
}
auto ret = squashing_transform.getFinalOutputBlock();
Block ret = getOutputBlock();
return ret;
}
catch (...)
Expand Down
1 change: 0 additions & 1 deletion dbms/src/DataStreams/HashJoinProbeBlockInputStream.h
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ class HashJoinProbeBlockInputStream : public IProfilingBlockInputStream
size_t probe_index;
ProbeProcessInfo probe_process_info;
BlockInputStreamPtr non_joined_stream;
SquashingHashJoinBlockTransform squashing_transform;
ProbeStatus status{ProbeStatus::PROBE};
size_t joined_rows = 0;
size_t non_joined_rows = 0;
Expand Down

0 comments on commit 2b1e538

Please sign in to comment.