-
Notifications
You must be signed in to change notification settings - Fork 11
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Wasm] Omit materialization in
wasm::SortMergeJoin
if possible.
Materialization can be omitted if the child operator is a table scan and no sorting has to be performed. In this case, we directly iterate over the underlying table store instead of allocating a buffer. Note that omitting materialization by reusing an already filled buffer is possible in general, however, our current implementation is restricted to passing only tuples to parent operators. Thus, this change was easy to implement and will benefit our benchmarks that assume an existing ordering on base relations.
- Loading branch information
1 parent
3c8b2dd
commit 1858b9c
Showing
1 changed file
with
64 additions
and
33 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters