Skip to content

Commit 9cd21ef

Browse files
committed
[Wasm] Remove hack to estimate initial hash table capacity.
For the `HashBasedGrouping` and `HashBasedGroupJoin` physical operators, the initializing hack does not work as only the groups are indexed but the number of tuples can be vastly larger.
1 parent 9cd4f4a commit 9cd21ef

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

src/backend/WasmOperator.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1472,8 +1472,6 @@ void HashBasedGrouping::execute(const Match<HashBasedGrouping> &M, setup_t setup
14721472
} else {
14731473
if (M.grouping.has_info())
14741474
initial_capacity = std::ceil(M.grouping.info().estimated_cardinality / M.load_factor);
1475-
else if (auto scan = cast<const ScanOperator>(M.grouping.child(0)))
1476-
initial_capacity = std::ceil(scan->store().num_rows() / M.load_factor);
14771475
else
14781476
initial_capacity = 1024; // fallback
14791477
}
@@ -4540,8 +4538,6 @@ void HashBasedGroupJoin::execute(const Match<HashBasedGroupJoin> &M, setup_t set
45404538
} else {
45414539
if (M.build.has_info())
45424540
initial_capacity = std::ceil(M.build.info().estimated_cardinality / M.load_factor);
4543-
else if (auto scan = cast<const ScanOperator>(&M.build))
4544-
initial_capacity = std::ceil(scan->store().num_rows() / M.load_factor);
45454541
else
45464542
initial_capacity = 1024; // fallback
45474543
}

0 commit comments

Comments
 (0)