Skip to content

Commit

Permalink
Address comment about adding description for the cardinality of each …
Browse files Browse the repository at this point in the history
…BatchState member
  • Loading branch information
zanmato1984 committed Feb 28, 2025
1 parent 30f6adf commit b2b67e4
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion cpp/src/arrow/acero/swiss_join_internal.h
Original file line number Diff line number Diff line change
Expand Up @@ -611,11 +611,17 @@ class SwissTableForJoinBuild {

// One per batch.
//
// Informations like hashes and partitions of each batch.
// Informations like hashes and partitions of each batch gathered in the partition phase
// and used in the build phase.
//
struct BatchState {
// Hashes for the batch, preserved in the partition phase to avoid recomputation in
// the build phase. One element per row in the batch.
std::vector<uint32_t> hashes;
// Accumulative number of rows in each partition for the batch. `num_prtns_` + 1
// elements.
std::vector<uint16_t> prtn_ranges;
// Row ids after partition sorting the batch. One element per row in the batch.
std::vector<uint16_t> prtn_row_ids;
};

Expand Down

0 comments on commit b2b67e4

Please sign in to comment.