JerAguilon commented on code in PR #41874:
URL: https://github.com/apache/arrow/pull/41874#discussion_r1619106973
##########
cpp/src/arrow/acero/asof_join_node.cc:
##########
@@ -912,7 +912,12 @@ class CompositeTableBuilder {
}
}
}
- return CompositeTable{schema, inputs.size(), dst_to_src, pool};
+ // NB: The left hand side of the join (index 0) is the reference table of
the
+ // asof-join. The output is comprised of continuous slices of the this
table, so
+ // we can just take zero-copy slices of it. This is much faster than how
other
+ // tables are treated, wherein we need to copy
+ return CompositeTable{schema, inputs.size(), dst_to_src, pool,
+ /*contiguous_sources=*/{0}};
Review Comment:
To aid review as this branch is rebased on top of
https://github.com/apache/arrow/pull/41125
This line is changed by this PR.
##########
cpp/src/arrow/acero/unmaterialized_table.h:
##########
@@ -18,10 +18,12 @@
#pragma once
#include <optional>
+#include <unordered_set>
Review Comment:
To aid review as this branch is rebased on top of
https://github.com/apache/arrow/pull/41125
This file is changed by this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
